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

Request: Add feedback option to Chorus to make a flanger #575

Closed
DannyDane opened this issue Nov 13, 2019 · 2 comments
Closed

Request: Add feedback option to Chorus to make a flanger #575

DannyDane opened this issue Nov 13, 2019 · 2 comments

Comments

@DannyDane
Copy link

Hi there,

As I understand, the only differences between a Chorus and a Flanger are these:

  1. A Flanger has a shorter delay time than a Chorus.

  2. The delay time of a Flanger modulates to 0, but doesn't with a Chorus.

  3. With a Flanger, the delay has feedback, with a chorus it doesn't.

You can do the first 2 with the current implementation, but not the 3rd.

Would it be possible to add a feedback option to the Chorus?
If it is as simple as swapping the Delay for a FeedbackDelay I could try it myself, although I'm not entirely familiar with Tone's architecture.

Or maybe make Flanger is own, separate effect. But I wouldn't know how to do that without copying alot of code from the Chorus effect.

Thanks in advance

@tambien
Copy link
Contributor

tambien commented Nov 13, 2019

I made Chorus extend StereoFeedbackEffect instead of StereoEffect which now gives it a feedback property. I test it out and with the right settings you get flanger-y effects. With chorus.feedback.value = 0 it is the same old chorus effect.

I'll also add a note in the docs which mentions that this effect can be used for both chorus and flanger effects

@DannyDane
Copy link
Author

That's awesome. Thanks tambien!

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