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

Stereo Chorus #120

Merged
merged 2 commits into from Jun 29, 2023
Merged

Conversation

alter-alter
Copy link
Contributor

Added a stereo chorus effect to the MOD FXs

@litui
Copy link
Collaborator

litui commented Jun 29, 2023

@alter-alter heya! Thanks very much for the submission! I'll run workflows and see if we can get this tested and in soon =)

@litui litui added the enhancement New feature or request label Jun 29, 2023
@litui
Copy link
Collaborator

litui commented Jun 29, 2023

Looks like just formatting failing. If you could give it a run with clang-format / clang-tidy 14.x that should take care of that.

What builds have been tested on the Deluge hardware so far?

@jamiefaye
Copy link
Collaborator

I did figure out where the Mod FX stuff was buried.

@m-m-adams
Copy link
Collaborator

Would it be possible to generalize this to add a second mod fx LFO at a slightly different rate and have it do stereo versions of all effects?

As is I think I hear phasing issues if I use the stereo chorus and then collapse to mono in a daw

@alter-alter
Copy link
Contributor Author

What builds have been tested on the Deluge hardware so far?

I have tested the release-OLED build.

@jamiefaye jamiefaye merged commit eff64a4 into SynthstromAudible:community Jun 29, 2023
5 checks passed
@alter-alter
Copy link
Contributor Author

As is I think I hear phasing issues if I use the stereo chorus and then collapse to mono in a daw

I think phase problems are unlikely to occur because the pitch changes with the modulation of the delay time. (When the right channel rises in pitch, the left channel falls.)

@litui
Copy link
Collaborator

litui commented Jun 29, 2023

Congrats on the merge! We appreciate your contribution back to the community build.

@alter-alter
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants