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

LFOEG Sustain Not Modulatable during Gate After Decay #858

Open
Andreya-Autumn opened this issue Feb 23, 2023 · 2 comments
Open

LFOEG Sustain Not Modulatable during Gate After Decay #858

Andreya-Autumn opened this issue Feb 23, 2023 · 2 comments

Comments

@Andreya-Autumn
Copy link
Collaborator

If changes and modulations to EG params were applied in continuously on the LFOxEG out, that would allow some cool modulations, plus make LFOxEG interactions easier to understand! For now, the EG params only update at each trig.

@baconpaul
Copy link
Contributor

Sigh. So the problem here is a bit more subtle.

The AHDRS are snapped when modulated and are passed in. you can see this by setting a long attack and modulating it with a high frequency lfo and it wiggles as it goes up

The problem is the surge LFO sustain phase gets to the release point at the end of the decay phase and holds that constant.

You can see this in the VST. If you set up an envelope and have LFO2 -> LFO1 sustain, apply LFO1 to pitch and have a long decay and a fast LFO2 you can hear the system wobbling as the decay happens - the target is shifting - but once the decay time ends and you get to the sustain the modulation doesn't impact the sustain curve.

We could change that but it's a breaking change for surge so needs a switch and so on.

So for now I'm going to rename this issue to be more accurate ("Sustain Not Modulatable when gated after decay") and move it out of 2.1.4.

@baconpaul baconpaul changed the title Update LFOxEG evenvelope parameters continuously LFOEG Sustain Not Modulatable during Gate After Decay Mar 4, 2023
@baconpaul baconpaul modified the milestones: Xt 2.1.4, Xt 2.2 Mar 4, 2023
@baconpaul
Copy link
Contributor

Ahh I should have looked at this earlier in S13 cycle; i don't want to screw with the LFO in surge proper so I'm going to knock this out of 22 milestone also. We should bring both issues into a current milestone earlier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants