-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
HUI Protocol Support #136
Comments
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Features
.lossy(String)
etc.)Fixes
HUICoreEvent.largeDisplay(slices:)
is open-ended array of chars. When sending this event, the MIDI event encoder could validate this to pad/trim to exactly 10 chars.Pre-Release
Fader
view to match dbVU scale that is marked on the HUI device surfaceDone
HUISurface
HUICoreEvent
is slightly abstracted but should it be?HUIEvent
(model change event) should carry these full strings, and when sending/receiving HUI only delta (partial) change messages should be sent when invokingtransmitLargeDisplay()
onHUIHostBank
HUICoreEvent
be internal and not public?translator: HUIModel
tomodel:
and make it public? Allow more interactivity with the model for the host bank? Tricky part is causing updates to the model to be transmit to the bank’s midiOut handler.HUISwitch.allCases
through aforEach {}
unit test to see if they all encode and decode successfullyLogger.debug()
- or have a Bool setting to enable logging? It only logs in DEBUG builds any way.Display
Deferred / Cancelled
HUISurface
full state push?(not to host of course, there's nothing to push. but to the handler that is subscribed to hui surface change notifications)
HUIHostBank
additions(to client surface, ie: when a client appears after a break in ping message stream (loss of surface presence))
The text was updated successfully, but these errors were encountered: