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.

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

[BUG] Output data is wrong when MT4 messages break up in the middle of a message. part.2 #458

Open
sat-okada opened this issue Dec 9, 2024 · 3 comments
Assignees
Labels
bug 🐞 Something isn't working

Comments

@sat-okada
Copy link
Collaborator

Describe the bug
MT4 messages consist of two words, but inputting one word at a time may result in incorrect output results.
Similar issue as #457.

To Reproduce

  1. Install zoa_odd.zip on the ProtoZOA.
  2. Start the console and monitor the ProtoZOA.
  3. Press TP6

Then receive the following messages:
Image

Expected behavior
The following messages should be received following Index 5:
0x40954500 0x80000006
0x40964600 0x80000007

Installer Name or Version
・Windows.MIDI.Services.In-Box.Service.-.1.0.1-preview.7.24305.1438-x64.exe
・Windows.MIDI.Services.Tools.and.SDKs.1.0.1-preview.7.24305.1438-x64.exe
・USB MIDI 2.0 class driver (USBMIDI2_10.0.1.7.x64.zip)

Desktop (please complete the following information):
・OS: Windows 11 Pro Insider Preview Build 27758.rs_prerelease.241122-1512

Device information, if this is with an external MIDI device:
ProtoZOA

Application Information
midi.exe console app.

Additional context
Attached firmware is same as #457.

@sat-okada sat-okada added the bug 🐞 Something isn't working label Dec 9, 2024
@sat-okada
Copy link
Collaborator Author

I checked the data on the USB for this case.

UMPpacket31 is the same size as UMPpacket30, missing 4 words.
Image

@Psychlist1972
Copy link
Contributor

inputting one word at a time may result in incorrect output results.

Can you explain this further, please? We only input and output complete UMP messages, and expect MIDI 2.0 devices to do the same. A partial UMP message is invalid data.

@sat-okada
Copy link
Collaborator Author

The purpose of this test is to check the processing of UMP messages across multiple USB packets in comment #312.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants