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.

Sign up for GitHub

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

Jump to bottom

Make all the random opcodes bipolar #452

Merged

jpcima merged 2 commits into sfztools:develop from jpcima:fil-random

Sep 27, 2020

Collaborator

jpcima commented Sep 27, 2020

Reported by @kinwie. It's an inconsistency between random opcodes.

apparently Cakewalk uses bipolar for everything
ARIA uses unipolar
bipolar may be considered as more useful, and probably it makes the most sense.

jpcima added 2 commits

September 27, 2020 15:37


          Make filN_random real-valued, and bipolar

283a7a3


          Also make pitch_random real-valued

afc0b3f

jpcima requested a review from paulfd

September 27, 2020 13:57

paulfd approved these changes

View reviewed changes

Member

paulfd commented Sep 27, 2020

Yes I think I did not think too much about the behavior at the time and I had only offset_random, which I guess should still be unipolar?

jpcima merged commit cf90794 into sfztools:develop

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

Labels

None yet