#14 'Mono' mode flaky implementation

Closed
opened 8 months ago by dronus · 7 comments
dronus commented 8 months ago

The MONO mode implementation seems to be incomplete.

-It is toggled by a MicroDexed-proprietary(?) CC on controller 126 and 127, while on the original DX7 it is toggled by a ParameterChange sysex Group 2, Parameter 64.

-There is no portamento, pitch changes are immediately and and may expose artifacts like clicking or crackling.

The MONO mode implementation seems to be incomplete. -It is toggled by a MicroDexed-proprietary(?) CC on controller 126 and 127, while on the original DX7 it is toggled by a ParameterChange sysex Group 2, Parameter 64. -There is no portamento, pitch changes are immediately and and may expose artifacts like clicking or crackling.
dronus commented 8 months ago
Poster

Actually the correct MIDI command is already mentioned by doc/sysex-format.txt

Actually the correct MIDI command is already mentioned by doc/sysex-format.txt
dcoredump commented 8 months ago
Owner

Ok, thanks for reporting. I will try to fix it asap.

Ok, thanks for reporting. I will try to fix it asap.
dcoredump commented 7 months ago
Owner

For the mono/poly-mode: I used https://nickfever.com/Music/midi-cc-list as reference and there was MIDI-CC 126/127 mentioned for setting mono/poly. I have to check if I can also add the “original” sysex-mode.

Portamento is currently not implemented. There is a pull request for the original dexed at https://github.com/asb2m10/dexed/pull/183. Perhaps this can be integrated into MixroDexed, too.

For the mono/poly-mode: I used https://nickfever.com/Music/midi-cc-list as reference and there was MIDI-CC 126/127 mentioned for setting mono/poly. I have to check if I can also add the "original" sysex-mode. Portamento is currently not implemented. There is a pull request for the original dexed at https://github.com/asb2m10/dexed/pull/183. Perhaps this can be integrated into MixroDexed, too.
dronus commented 7 months ago
Poster

Maybe this should be solved upstream at Dexed first... It is actually the largest gap in Dexeds emulation of the DX7.

Maybe this should be solved upstream at Dexed first... It is actually the largest gap in Dexeds emulation of the DX7.
dcoredump commented 7 months ago
Owner

I think upstream integration will take some time, perhaps I will try to merge before...

I think upstream integration will take some time, perhaps I will try to merge before...
dcoredump commented 7 months ago
Owner

I have added “ParameterChange sysex Group 2, Parameter 64” (inside dev-branch) - hopefully it will work. Can you check this?

I have added "ParameterChange sysex Group 2, Parameter 64" (inside dev-branch) - hopefully it will work. Can you check this?
dcoredump commented 1 month ago
Owner

Hi @dronus,

the protamento code is inside the dev-branch. Only glissando seems to have a small problem.

Regards, Holger

Hi @dronus, the protamento code is inside the dev-branch. Only glissando seems to have a small problem. Regards, Holger
dcoredump closed this issue 1 month ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.