First commit of proposed 00 Simple MIDI v2 - #129
Conversation
|
Version for testing at https://chris-j.co.uk/misc/simple_midi_2/ with firmware https://chris-j.co.uk/misc/simple_midi_2/simple_midi_2.uf2 . I'll try to keep these up to date with the version here. |
|
I'm leaving as a draft PR for now as this is not 'my' card to edit. Everything is in a |
Lots of bug fixes: - MIDI pitch --> CV outputs now work even if no calibration saved - Fixed Bernoulli gate triggered by pulse in or MIDI gate - Fixed Turing machine clock divider when triggered by pulse in - Fixed reverb freeze driven by MIDI note - Fixed board ID bug - Fixed MIDI bug where both sysex and non-sysex in one USB packet - Fixed bug where note on with velocity 0 counted as trigger for Turing Machine - Fixed CV out (with divider) triggered by pulse in Improvements to UI: - Removed dependency on WebMIDI.js, in so doing fixing 100% CPU usage in Chrome if MIDI connection removed - Populated <option> tags through JS - Re-compressed base64-encoded PNGs - Horizontal rules now separate sections of the sentence UI - Result of these is to reduce HTML size from ~250KB to ~50KB.
|
Finally after so long got round to testing this. Worked exactly as expected, hosting worked first time with an elderly keystep and KO-2 Nice to have pitchbend! Calibration also went very smoothly - the only UX issue I had - and it's very small - after the calibration I get a graph and it says ±1 cent - is this OK? I had no idea what normal would mean. I feel if you're still happy with it that we should:
|
|
Thanks Tom, Calibration UI - yes I also had no idea what the accuracy was going to be, but most people are between -1 and +1 cent, except at the lowest octave when it tends to drop to more like -2 cents. I think it needs a sentence to that effect, which I'll add. Full disclaimer before we merge:
Mitigating most of these concerns is that almost all the calibration logic is in the web UI, not the card itself, and so even if we did fix/improve any of these issues later, it'd probably be an update to the website only. |
✅ Program card PR validation succeededPR validation is intended for maintainers, not card authors. If you're a card author and don't understand this report, don't worry about it! 1 info.yaml file(s) checked · 0 error(s) · 0 warning(s) Triggered by:
Affected release directories: ✅ All changed Other rules✅ All submission rules passed. |
First commit of a prototype ComputerCard-based
00 Simple MIDIcard, put in av2_prototypesubdirectory of this card.New features:
Remaining TODOs: