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

Send volume changes to controller #310

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

wvengen
Copy link
Contributor

@wvengen wvengen commented Jun 12, 2020

Ad this moment, the volume sliders can not send midi changes. This is relevant for controllers with motorized faders and led-based (rotary or fader) controls.

Some controllers can also gradually change the volume, which doesn't work when volume changes sent by the controller are sent back to it. That's why this needs to be controlled as well.

This PR adds midi messages for all volume/pan/send midi events, and the ability to toggle echo for these messages. Tested on the Launchpad X (#309).

  • Code-style consistency, use Event::SOURCE or SOURCE (and the enum constants).

@wvengen wvengen mentioned this pull request Jun 17, 2020
@harryhaaren
Copy link
Member

@wvengen, any opens around this? Anything I can do in 30 mins to move it along? (And my apologies that I haven't been working on this stuff at all in recent months - but that's real life for me atm : )

@wvengen
Copy link
Contributor Author

wvengen commented Jun 14, 2021

Hi @harryhaaren thank you for your response! I'd love to get the PRs merged (even if it will take a while, also from my side).
If you could take a first look at the code and indicate the most important things to address, that would help. And also hear if you'd prefer the use of Event::SOURCE or SOURCE.

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

Successfully merging this pull request may close these issues.

None yet

2 participants