Skip to content

rework convolution node#123

Merged
BillyDM merged 3 commits intomainfrom
rework_convolution
Mar 21, 2026
Merged

rework convolution node#123
BillyDM merged 3 commits intomainfrom
rework_convolution

Conversation

@BillyDM
Copy link
Copy Markdown
Owner

@BillyDM BillyDM commented Mar 20, 2026

The convolution node has been reworked:

  • updated to the latest version of fft-convolver
  • made the impulse response a parameter, similar to how the sample is a parameter in the sampler node
  • removed the const CHANNEL generic, and also added support for convolution nodes with an arbitrary number of channels.

However, I'm getting a clicking noise when switching impulses. I'll have to debug that.

@BillyDM
Copy link
Copy Markdown
Owner Author

BillyDM commented Mar 21, 2026

Ok I guess FFTConvolver::set_response() isn't actually seamless. Darn, guess I have to go back to using two convolvers and fade between them.

@BillyDM
Copy link
Copy Markdown
Owner Author

BillyDM commented Mar 21, 2026

Ok, I managed to fix the declicking by just fading out the old impulse first before replacing it with the new one.

@BillyDM BillyDM merged commit 3eee263 into main Mar 21, 2026
3 checks passed
@BillyDM BillyDM deleted the rework_convolution branch March 21, 2026 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant