MIDI UART support

[CedarGroveStudios]

Would it be possible to expand this to also support "classic" MIDI communications via a UART port?

[ladyada]

im cool with it if you can figure out how :)

[CedarGroveStudios]

Thanks. "I'll take a shot at it," he said nervously.

[CedarGroveStudios]

Status: With just a few changes I was able to create a working UART-only version, but I'm not sure how to combine the two and maintain the original UI -- just yet. I'm looking at other libraries for similar examples and plan to talk to some of the folks on the CPy team for advice. This might take me a while... but I'm having fun.

[tannewt]

@CedarGroveStudios Can you post your changes somewhere? The goal of the USB midi stuff was to act exactly like a UART so this lib would work with both. Maybe something needs to change with it.

[CedarGroveStudios]

@tannewt That was my objective, too. I have fork of the repo with a MIDI_UART class (and the original but renamed MIDI_USB class) and updated examples. Should I just create a PR from the updated fork? (I'm so new at using GitHub for this.)

[CedarGroveStudios]

The fork is here: https://github.com/CedarGroveStudios/Adafruit_CircuitPython_MIDI

[tannewt]

@CedarGroveStudios Could you rename the uart version back to the original filename so that the diff is shown? It looks to me like the only change is taking in midi_in and midi_out as pins instead of serial objects.

Did you try giving a UART as both midi_in and midi_out? I'd expect that to just work.

[CedarGroveStudios]

@tannewt The USB version doesn't handle midi_in if I'm understanding it correctly, so I just duplicated the same output-only functionality by adding the UART pins and unique baud/timeout settings as you noted. I created a second class to facilitate instantiation. Is there a more elegant way of doing it that still allows the creation of a UART instance?

from adafruit_midi import midi_uart

# defaults to transmit out the board.TX pin
midi = midi_uart.MIDI(out_channel=0)

The USB midi_out uses an abbreviated subset of the MIDI protocol stack (CC, Note_On, etc.). Parsing the USB and UART midi_in messages using a similar subset of the MIDI protocol stack would be something for the next version, I'd guess. For this I was just trying to duplicate the existing functionality of the USB code.

[tannewt]

I was thinking:

import board
import busio
import adafruit_midi

uart = busio.UART(board.TX, rx=None, baudrate=31250, timeout=0.001)
midi = adafruit_midi.MIDI(midi_out=uart, out_channel=0)

Check with @kevinjwalters about MIDI in. I think they've done some work on it already.

[CedarGroveStudios]

^ That's an excellent approach that could lead to using any MIDI input/output stream including USB, UART, BLE, and file. In your example, how would USB or BLE in/out be instantiated?
I just looked at @kevinjwalters' repo. Their approach is appropriately MIDI message-centric and could be applied to the midi_out functionality as well. Looks to me like a better way to implement than the one I had in mind.

[tannewt]

We don't have BLE support yet.

USB is already instantiated because the USB descriptor is fixed. It is available as usb_midi.ports[0] or usb_midi.ports[1]. You can check direction by checking the object type.

It's a bit weird that those are the default values.

[kevinjwalters]

FYI, I'm hoping to have the MIDI stuff done by Sunday and will look at allowing messages to be sent whilst preserving the existing interface. I'm trying to keep it reasonably tight on memory which is my current area of exploration.

[CedarGroveStudios]

BTW, I successfully tested USB inputs and outputs along with simultaneous UART output using the existing library. My DAW calls the GrandCentral's USB port an Adafruit CircuitPython keyboard. Nice.
Since I already have UART input/output code working for the MIDI protocol stack as part of my Crunchable synth project, I'll continue working on converting it to a library. As @kevinjwalters noted, it's a challenge to fit it into memory (it's a protocol stack approach rather than message-centric). However, I think it's doable.
I'm a primarily a hardware person so my coding efforts aren't going to win any races. Don't wait for me.

[kevinjwalters]

WRT to comment on default values for midi_in and midi_out, there are now no defaults as of #11