-
Notifications
You must be signed in to change notification settings - Fork 78
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
Support more MIDI CCs #122
Conversation
Closes #121
I think I was able to implement them. Except for 91 = Reverb Level, because there is no |
Regarding 91 (Reverb Level) I'm honestly not sure. Perhaps it's better to use
Some parameters are not in the MIDI range 0..127. They must be mapped to that range. |
Yes, it'd be more consistent with the other params indeed.
Line 386 in 43d1b3a
should take care of that, right? |
|
10 = Pan: 0-127 |
Add support for MIDI CCs
Closes #121