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

Bipolar mode changes display : Feature request #129

Open
muziker opened this issue Jan 8, 2021 · 1 comment
Open

Bipolar mode changes display : Feature request #129

muziker opened this issue Jan 8, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@muziker
Copy link

muziker commented Jan 8, 2021

It looks as if some other wave shapers in bipolar mode have a center point at the zero crossing mark, and converts the unipolar mode graph into the positive part of the graph, and a linear graph onto the negative part. Right now, the graph is used as-is in bipolar mode when switched from unipolar mode. Maybe the bi/uni button should also include this behaviour of having a center zeroed keypoint when switched

@pdesaulniers pdesaulniers added the enhancement New feature or request label Jan 8, 2021
@pdesaulniers
Copy link
Member

pdesaulniers commented Jan 8, 2021

Yes, I think that would make sense. However, instead of setting a linear function on the negative part, I would simply convert the unipolar graph into an equivalent bipolar graph. That way, activating bipolar mode would not affect the output (until the user tweaks the function).

Alternatively, we could reimplement unipolar mode so that it displays both the negative and positive part of the function simultaneously. Every modification to the graph on the positive side would be mirrored on the negative side, and vice versa. In bipolar mode, both sides of the graph would be editable independently (activating bipolar mode would simply disable the automatic "mirroring" of the graph). The downside of this approach is that it wastes a bit of screen space in unipolar mode, since half of the graph would now show duplicate information.

I believe @unfa proposed this feature a few years ago 😛 Related issue: #68

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

No branches or pull requests

2 participants