Oversampling on convolution reverb
-
@lalalandsynth no aliasing on 0oura but IPP
Don't know details about the comments on the code I posted. It's old @Christoph-Hart magic! :)
-
@hisefilo ok, but it does not seem to work for me , lose the reverb sound if I change the FFT mode.
I am going to test some more impulses , but I think its pretty clear that there is a problem.
My reverb plugin is just about ready :( -
Any chance to get a fix for this?
Was planning an impulse reverb as my first release. :) -
@lalalandsynth an oversampled convolution node on scriptnode?
-
@hisefilo Oh, that is possible , does it behave in a similar way in terms of loading impulses etc or do I have to do everything over ?
Will check it out.
-
@lalalandsynth Quick test and that seems to work to get rid of the aliasing!
8x oversampling, maybe even 2 x would be enough , now I just need to find out if I can replace those impulses on the fly or if the scriptnode impulse is meant to just sit there as an static impulse ?I suspect i cannot even get a reference for the Convolution Module in the scriptnode, is there some way to script the switching of impulses in scriptnode ?
I can export the Convolution module as ccp and make the changing of impulses public i.e create a meta node but unsure of how to include the oversampling in that ?
Man..if there was only a knob for the index.....sigh .
-
Check this out , seems to work .
https://forum.hise.audio/topic/2652/scriptnode-oversample-node-starting-at-4x-crashes-in-pc/13 -
BTW, what does the "multithread option" do on the Convolution reverb ?
Its not available on the Scriptnode Convolution so just wondering. -
It moves the convolution calculation into a background thread. It's not faster in anyway, but it won't increase the CPU usage of your audio thread, so if your main goal is to keep the CPU meter down, this is the key.
-
@lalalandsynth Any progress on the aliasing? I'm experiencing the same issue. Setting the FFTType to 3 fixes it mostly, but also crashes my plugin and removes the reverb when MultiThread is active, which defeats the whole purpose. This is really making the convolution almost unusable.
How have other people achieved good results with the convolution module so far? It seems almost too whack for production-ready plugins.
-
@Lunacy-Audio I have it working in Scriptnode wrapped in oversampling, no multithread option though.
Just using a delay with no feedback as the predelay.However , you cannot switch impulses without major clicks and pops.
So yeah , not ready for plugins in my opinion.
-
Hmm bummer. Convolution is SO important!
I just noticed that if I disable IPP in the HISE build, the artifacts go away and you can keep MultiThread active. I might try this option with my test users and see if any performance issues arise. -
@Casey-Kolb Just to follow up, are you able to fix this convolution issue?
Is disabling IPP helped?As I see, for this issue, the IR length doesn't matters.
-
@orange We ended up just disabling IPP completely and that fixed the issue. No serious issues in terms of performance so far, but it would be nice to have IPP at some point without this issue.
-
@Casey-Kolb I really don't want to disable IPP, but if I'll have to, I can live with it.
-
@orange Are you noticing the artifacts with IPP active?
-
@Casey-Kolb Yes, when IPP is deactive the artifacts are gone
