Wohlstand / OPN2BankEditor

A small cross-platform editor of the OPN2 FM banks of different formats (Downloads in README below)
GNU General Public License v3.0
40 stars 8 forks source link

Bad definition for frequency multiplier MUL #21

Closed jpcima closed 6 years ago

jpcima commented 6 years ago

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 commented 6 years ago

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

freq-mod commented 6 years ago

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 commented 6 years ago

Fixed on a quick hand!