LONPRESSLATCHING and POWERWINDOW is not working from midi

Bug #1277449 reported by Daniel Schürmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Medium
Daniel Schürmann

Bug Description

The code inside Mixxx assumes that the Buttons on the Midi device will send
MIDI_NOTE_ON and
MIDI_NOTE_OFF

At Least the RMX2 controller sends allays MIDI_NOTE_ON

Debug [Controller]: "MIDI status 0x90 (ch 1, opcode 0x9), ctrl 0x23, val 0x7F"
Debug [Controller]: "MIDI status 0x90 (ch 1, opcode 0x9), ctrl 0x23, val 0x00"

Inside the code there is a comment:
// looking for NOTE_ON doesn't seem to work...
This and the other !value toggle workarounds are looking like all controllers are sending a good value and we can ignore the MIDI_NOTE.

If this is not the case we have to make it configurable.

Changed in mixxx:
assignee: nobody → Daniel Schürmann (daschuer)
importance: Undecided → Medium
status: New → Confirmed
milestone: none → 1.12.0
Revision history for this message
Daniel Schürmann (daschuer) wrote :

Ok I was wrong. From the log of a AmericanAudio VMS4 in Bug #1277391 we see:

Debug [Controller]: "MIDI status 0x90 (ch 1, opcode 0x9), ctrl 0x0D, val 0x7F"
Debug [Controller]: "MIDI status 0x80 (ch 1, opcode 0x8), ctrl 0x0D, val 0x40"

Revision history for this message
Daniel Schürmann (daschuer) wrote :
Changed in mixxx:
status: Confirmed → In Progress
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Always sending note-on messages means the MIDI controller has broken or stupid firmware :) -- I'm against hacking Mixxx for this. We used to do this with MIDI Option flags but adding more and more of those became unmaintainable. This should be solved with MIDI script.

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Hm, I read the linked article: http://de.wikipedia.org/wiki/Musical_Instrument_Digital_Interface and some other documents and it does seem a note-on with velocity of 0 could be interpreted as a note-off. Sorry about that!

Changed in mixxx:
status: In Progress → Fix Committed
RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7297

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.