High-pitched noise with RR group XFades with low buffer sizes

[Christoph Hart]

@tomekslesicki you can just change the location I modified in the commit and use an even smaller number above zero, then let me know what you end up with :)

[tomekslesicki]

@Christoph-Hart well, that was a logical answer :D I built a couple of versions with different values and 0.00000000001f works prefect for all buffer sizes.

[d.healey]

@tomekslesicki said in High-pitched noise with RR group XFades with low buffer sizes:

0.00000000001f

That's some precision!

[Christoph Hart]

0.00000000001f

haha this is basically zero with the single precision resolution of seven digits... You're effectively just deactivating the logic that will save you CPU cycles when there is nothing going on in the modulation signal.

I guess I'll rather make a preprocessor like HISE_ENABLE_CROSSFADE_MODULATION_THRESHOLD and if that is set to true, it uses the -80dB threshold that gets rid of most artifacts and if that's not enough for your particular project you can disable it entirely by setting this to zero.

[tomekslesicki]

@Christoph-Hart sounds like a good plan! ;-) By setting it to 0, you mean something like this?

HISE_ENABLE_CROSSFADE_MODULATION_THRESHOLD=0

[Christoph Hart]

@tomekslesicki yes. Also you then can just leave it as default in HISE and just add that to your project definitions (if you can live with the low artifacts at -80dB during development), this way you don't have to edit the source code at all.

[tomekslesicki]

@Christoph-Hart epic, thank you!

[tomekslesicki]

@Christoph-Hart we don't have this HISE_ENABLE_CROSSFADE_MODULATION_THRESHOLD yet, right?

[Christoph Hart]

@tomekslesicki ah yes my goldfish brain with its limited attention span moved on to the next fancy thing…

[tomekslesicki]

@Christoph-Hart if I could then send a little wish to that goldfish, I'd say that HISE_ENABLE_CROSSFADE_MODULATION_THRESHOLD would make me very happy ;-)

[d.healey]

@tomekslesicki If the goldfish can grant wishes then I wish for more goldfish!

[Christoph Hart]

@tomekslesicki There you go...