Clearly document panic cases

[kangalio]

No description provided.

[lucemans]

Im wondering if there is a specific reason for panicking when encountering an unexpected midi value

launchy/src/launchpad_mini/input.rs

Line 71 in 3e1510d

other => panic!("Unexpected midi message: {:?}", other),

For context: this is the launchpad mini (mk1), with the example version inquiry code from readme

[lucemans]

Hmm it appears [128, x, 64] is sent by the launchpad on release of a button, where X = 0 for the most topleft gridbutton
x = 16 for the first button the row below, etc

[kangalio]

Cc @belak who contributed Launchpad Mini support in #3

Yeah 128 = 0x80 is the status code for a MIDI note off message. In the code it explained there is no handler for note off messages because the Launchpad doesn't use it... which is evidently false in this case

[lucemans]

Happy to make a PR where possible. I currently have a semi-patched version on lucemans/launchy.

Also working on mini mk3 support, but after some testing it has proven to be a lot more difficult seeing it has "scenes" and ableton specific things built in by default.

[lucemans]

Created an issue with my findings so far related to the mini mk3

Looking forward to working on this!
#12

[belak]

Interesting. I'm not sure why I didn't run into this when I was working on LP Mini support. I can try and take a look in the next few days.