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

Bad definition for frequency multiplier MUL #21

Closed
jpcima opened this issue Jun 20, 2018 · 3 comments
Closed

Bad definition for frequency multiplier MUL #21

jpcima opened this issue Jun 20, 2018 · 3 comments
Assignees

Comments

@jpcima
Copy link
Collaborator

jpcima commented Jun 20, 2018

The info tip of MUL refers to the OPL3 sample rate in its formula. This can't be right.
Should this be instead 53267?

opn2mul

@Wohlstand
Copy link
Owner

I think, I need to check, this formula was copied from OPL3 documentation, for OPN2 it would be different, but yeah, 53267

@freq-mod
Copy link
Contributor

freq-mod commented Jul 5, 2018

Rate scale level description also seems misleading, rather than attenuating output towards higher pitches it shortens the envelope (entirely) - attack rates become faster, decays are shortened as well.

Think of it like it's OPL3's KSR.

@Wohlstand
Copy link
Owner

Fixed on a quick hand!

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

3 participants