Skip to content

unison stereo spread#223

Merged
jamiefaye merged 1 commit intoSynthstromAudible:communityfrom
bfredl:stereospread
Jul 24, 2023
Merged

unison stereo spread#223
jamiefaye merged 1 commit intoSynthstromAudible:communityfrom
bfredl:stereospread

Conversation

@bfredl
Copy link
Copy Markdown
Collaborator

@bfredl bfredl commented Jul 21, 2023

only works for basic waves (and wavetables) yet, not samples nor FM/RM. I'm gonna try refactor the mono vs stereo rendering logic, to not duplicate the same panning logic for samples, FM and ringmod.

For now the shortcut is to press SELECT inside the unison number menu, but we could also take the free spot above it (right of decimation).

@bfredl bfredl force-pushed the stereospread branch 5 times, most recently from d5d1a3a to 1f9d4f9 Compare July 23, 2023 14:59
@bfredl bfredl changed the title [DRAFT] unison stereo spread unison stereo spread Jul 23, 2023
For now the shortcut is to press SELECT inside the unison number menu,
but we could also take the free spot above it (right of decimation).
@bfredl bfredl marked this pull request as ready for review July 23, 2023 15:01
@bfredl
Copy link
Copy Markdown
Collaborator Author

bfredl commented Jul 23, 2023

Now it supports all synth types. Code could be simplified if renderBasicSource always took a StereoSample * buffer but I didn't want to change the flow of the algorithm yet for the case when this new functionality is turned off, without proper measurement of impact.

Note: the diff is slightly less scary than it looks since a lot of it is moving the code for stereo voice buffer from "subtractive" to the end of voice::render so it could be used by the other engines as well when needed.

@jamiefaye jamiefaye added this pull request to the merge queue Jul 24, 2023
Merged via the queue into SynthstromAudible:community with commit d6e741e Jul 24, 2023
@entzmingerc
Copy link
Copy Markdown
Contributor

entzmingerc commented Feb 11, 2025

Hey @bfredl, when I had oscillator sync on and I turn unison stereo spread to a nonzero value, the oscillator sync no longer applies. You can hear the sync happening with osc sync on, turning up the volume of oscillator 2, and modulating osc 2 transpose with an LFO and hear it the sync effects. Then with unison voice number set to 2 or more, when you turn up the unison stereo spread, the syncing effect is no longer heard. I'm on FW 1.2 but I believe this behavior existed with previous firmware.

I wasn't sure if this was a bug or if it's a limitation of the engine.

Also I think I've used unison stereo spread on 99% of synth patches I've made since you added this. Thank you very much for your work on Deluge.

@bfredl
Copy link
Copy Markdown
Collaborator Author

bfredl commented Feb 11, 2025

@entzmingerc sounds like a bug. I'd suggest open a separate issue so it can be tracked properly.

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.

3 participants