Skip to content
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

Extend range of LFO rate #6575

Open
RCJacH opened this issue Aug 27, 2022 · 2 comments
Open

Extend range of LFO rate #6575

RCJacH opened this issue Aug 27, 2022 · 2 comments
Labels
Breaking Change In order to close this issue we need to make a breaking change (no backwards compatibility) Feature Request New feature request Modulation Modulation related issues
Milestone

Comments

@RCJacH
Copy link

RCJacH commented Aug 27, 2022

Currently the maximum rate setting for a LFO is 512 Hz

max_rate_512hz

There are some techniques in sound design which uses very fast random LFO generators to make oscillator sound fatter than just with detuning.

Can we double the rate range? Or as an alternative, can we include an "Extend range" option just like with the Pitch parameter?

@RCJacH RCJacH added the Feature Request New feature request label Aug 27, 2022
@mkruselj
Copy link
Collaborator

mkruselj commented Aug 27, 2022

Currently, Surge's modulation system works in blocks of 32 samples. This means that control rate is, at 48k, 48000/32=1500 Hz, with Nyquist limit at 750 Hz. But you will notice that due to linear interpolation between two 32 sample blocks, LFOs in Surge actually start aliasing sooner than that frequency. So. adding Extend range option is kinda useless how things stand today.

This is not an easy change, and it would considerably increase the CPU usage of Surge. We may attempt to speed up the control rate to 16 samples in XT2, maybe even 8 samples. But there is no concrete timeline for that release.

At any rate don't expect per sample modulation processing in Surge. This would be quite CPU intensive, as you can glean from Odin 2, for example.

@mkruselj mkruselj added the Modulation Modulation related issues label Aug 27, 2022
@mkruselj mkruselj added this to the Surge XT 2.0 milestone Aug 27, 2022
@RCJacH
Copy link
Author

RCJacH commented Aug 27, 2022

Okay that's understandable, thank you for the explanation.

@mkruselj mkruselj added CLAP Issues related to CLAP plugin format Breaking Change In order to close this issue we need to make a breaking change (no backwards compatibility) and removed CLAP Issues related to CLAP plugin format labels Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change In order to close this issue we need to make a breaking change (no backwards compatibility) Feature Request New feature request Modulation Modulation related issues
Projects
None yet
Development

No branches or pull requests

2 participants