Skip to content
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.

Sign up for GitHub

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

Jump to bottom

MIKMIDI fails to receive sysex messages (sending works) #233

Closed
armadsen opened this issue Feb 8, 2018 · 7 comments
Closed

MIKMIDI fails to receive sysex messages (sending works) #233

armadsen opened this issue Feb 8, 2018 · 7 comments
Assignees
@armadsen
Labels
bug
Milestone
1.7

Comments

@armadsen
Copy link
Member

armadsen commented Feb 8, 2018

No description provided.

@armadsen armadsen self-assigned this Feb 8, 2018
@armadsen armadsen added the bug label Feb 8, 2018
@armadsen armadsen added this to the 1.6.3 milestone Feb 8, 2018
armadsen added a commit that referenced this issue Feb 8, 2018
@armadsen
Issue #233: Add failing test for failure to receive Sysex messages
98b541e
@armadsen
Copy link
Member Author

armadsen commented Feb 8, 2018

Failing test is now there in the form of +[MIKMIDICommandTests testCreatingSysexFromMIDIPacket].

armadsen added a commit that referenced this issue Feb 8, 2018
@armadsen
Issue #233: Fix parsing SysEx messages out of MIDIPacket in +[MIKMIDI…
a1ee3a2 
…Command commandsWithMIDIPacket:]
@armadsen
Copy link
Member Author

armadsen commented Feb 8, 2018

Should be fixed now, though I don't have any devices that send SysEx handy at the moment, so needs more testing.

@armadsen armadsen mentioned this issue Feb 8, 2018
Closed
@latenitefilms
Copy link

@armadsen - Unfortunately, as far as I can tell, I'm still not getting any MIDI callbacks from MIKMIDI, even though I'm seeing data flying around in MIDI Monitor.

screen shot 2018-02-08 at 11 05 42 pm

@latenitefilms
Copy link

FYI - I'm not seeing any of these hit MIKMIDI for whatever reason:

screen shot 2018-02-08 at 11 18 14 pm

@armadsen armadsen mentioned this issue Feb 9, 2018
Closed
@armadsen
Copy link
Member Author

I believe this is fixed. Let me know if you don't think so.

@latenitefilms
Copy link

@armadsen - Will do, thank you!!

@armadsen armadsen mentioned this issue Mar 17, 2018
Merged
@armadsen
Copy link
Member Author

Fixed by #243.

@armadsen armadsen modified the milestones: 1.6.3, 1.7 Mar 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@armadsen armadsen

Labels
bug
Projects
None yet
Milestone
1.7
Development

No branches or pull requests
2 participants
@armadsen @latenitefilms