Closed anzbert closed 2 years ago
Merged, with the change that pitch range is now -1 to 1.
Thx ! -1 to 1 is a good idea too :)
I was thinking about how to implement MPE messages. I may read the spec and see how I go.
On Fri, 11 Mar 2022, 15:58 Morten Boye Mortensen, @.***> wrote:
Merged, with the change that pitch range is now -1 to 1.
— Reply to this email directly, view it on GitHub https://github.com/InvisibleWrench/FlutterMidiCommand/pull/52#issuecomment-1064864362, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJIGOFA7KVPKIJHYMZCBOS3U7L4ILANCNFSM5QI4LG3Q . You are receiving this because you authored the thread.Message ID: @.***>
I've added a few more Midi message classes. I hope that is what you had in mind and you like the implementation. The Pitchbend is a bit opinionated. Ranging from 0-1 and pitch center being 0.5. Not sure if you agree with that. Anyway, I hope I could help again!
I think I am getting the hang of submitting proper pull requests now :)