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

synth: receive/send dx7 params as sysex #2093

Open
wants to merge 2 commits into
base: community
Choose a base branch
from

Conversation

bfredl
Copy link
Collaborator

@bfredl bfredl commented Jun 4, 2024

This works to edit a dx7 patch on the deluge using e.g. the Dexed desktop app, although it is a bit of a mess to setup.

Copy link
Contributor

github-actions bot commented Jun 4, 2024

Test Results

107 tests  ±0   107 ✅ ±0   0s ⏱️ ±0s
 16 suites ±0     0 💤 ±0 
 16 files   ±0     0 ❌ ±0 

Results for commit 0c781f0. ± Comparison against base commit 1f82d07.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@sapphire-arches sapphire-arches left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be good to add docs on how to set this up to the dx7 markdown.

@seangoodvibes seangoodvibes changed the title synth: receive/send dx7 params as sysex [DRAFT] synth: receive/send dx7 params as sysex Aug 5, 2024
@bfredl bfredl force-pushed the dx7midi branch 3 times, most recently from 2d5c258 to e4b85a7 Compare September 7, 2024 12:02
@bfredl bfredl marked this pull request as ready for review September 7, 2024 12:03
@seangoodvibes seangoodvibes changed the title [DRAFT] synth: receive/send dx7 params as sysex synth: receive/send dx7 params as sysex Sep 13, 2024
@stellar-aria stellar-aria added bug Something isn't working needs testing and removed bug Something isn't working labels Oct 6, 2024
- Connect the Deluge to the computer with USB
- Start dexed in "Standalone" mode.
- Click the "options" button in the top left.
- Make sure "Deluge MIDI 1" is selected as an "Active MIDI input". Also set "MIDI output" to "Deluge MIDI 1"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these instructions should probably go to midi port 3 - we don't implement midi sysex stream interruption so I think it could cause bugs if you're playing notes and editing at the same time

@m-m-adams
Copy link
Collaborator

This has a bunch of conflicts now - are you still working on it?

@bfredl
Copy link
Collaborator Author

bfredl commented Dec 28, 2024

Rebased and changed preferred sysex port to "Deluge MIDI 3"

@mweigt
Copy link
Contributor

mweigt commented Jan 6, 2025

First of all: Thank you for the fantastic Work! Enabling DX7 emulation on Deluge is an incredible added value!

As testing was requested: Just jammed for one hour using dexed as sound control surface while playing a track on the deluge. Everything seems to work.

Some nits :

  • I have to actually enter a single parameter sub-menu (i.e. "algorithm" or "sync") for the "learn" assign midi channel to work, not just the "DX7 parameters" menu. That might be the intended behaviour, but the Doc is a bit ambiguous here.
  • I have to activate midi follow mode for the deluge to react to dexed param changes. Again, this is probably the (currently) intended behaviour, but the docs could be interpreted as this being optional.
  • Is there/will there be a way to assign a fixed channel without using midi follow?
  • When using dexed to control the deluge, only one paramter seems to be sent at each edit: switching to a different patch on dexed is not reflected on the deluge. I do not know if dexed is not sending the data or deluge not recieving/processing this.
  • Is there/will there be a way to additionally learn single parameters to normal midi CCs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants