-
-
Notifications
You must be signed in to change notification settings - Fork 122
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for using samples as impulse response buffers for the reverb #717
Conversation
after the first reverb PR is now merged, it seems it has created a bunch of conflicts here.. should I try to resolve them or do you want? :P |
if I'm not mistaken, right now this is just formatting in reverbGen file, and the rest is just my additions |
I had to do another merge, where I also removed the setIR function. it still seems to work as intended or did I remove something I shouldn't have? |
setIR was indeed redundant I guess, but there's a problem now with getting a url out of an object. I get this when I try to use samples like 'ocarina' or 'harmonica'
|
works like this, not sure if this is as it was before.. |
yeah, everything seems to work great! |
so ready for merge? |
I think so, yeah |
This PR adds a function
ir()
that accepts the names of samples to load as an impulse response for the reverb. thesize()
function creates a new buffer of the requested size.This also works:
If the
ir()
function is not applied, the regular impulse response generation is used.