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

Add MIDI channel to control changes dialog #156

Open
lazzarello opened this issue Jan 25, 2022 · 2 comments
Open

Add MIDI channel to control changes dialog #156

lazzarello opened this issue Jan 25, 2022 · 2 comments
Labels
feature request Request to add or idea for a new feature or enhancment of an existing one

Comments

@lazzarello
Copy link

I have a Korg nanoKontrol hardware surface. It has a "mixer mode" preset in hardware where all the 9 faders transmit CC 7 on channels 1-9. This doesn't work with jack mixer, so adding a "MIDI channel" option to the control changes dialog seems like a reasonable compromise.

Modifying the dynamic MIDI learn system seems more complicated, though it would be nice if it stored the channel byte and the CC byte when the hardware device did a wiggle.

I'm new to cython but I'm hacking my way through UI changes right now.

@SpotlightKid SpotlightKid added the feature request Request to add or idea for a new feature or enhancment of an existing one label Jan 25, 2022
@SpotlightKid
Copy link
Contributor

Thanks for the suggestion. The MIDI control feature needs a general overhaul. I'd like for jack_mixer to be able to be controlled via all types of MIDI messages, with configurable channels and value thresholds (e.g. for on/off elements).

I'll look favourably on a PR, which adds MIDI channel awareness, but this whole functionality might be reworked again in the midterm.

@lazzarello
Copy link
Author

lazzarello commented Jan 25, 2022

Thanks for the reply. I made some progress last night. I have a GUI element for channel like so

image

and I think I hit the cython wall because I can't get rid of this exception when adding the channel. Here's my PR, which is not finished.

lee@wendy:~/funsrc/jack_mixer$ ./tools/jack_mixer.sh 
Traceback (most recent call last):
  File "/home/lee/funsrc/jack_mixer/jack_mixer/app.py", line 804, in on_add_input_channel
    return self.on_add_channel("input", _("Input"))
  File "/home/lee/funsrc/jack_mixer/jack_mixer/app.py", line 799, in on_add_channel
    (self.add_channel if inout == "input" else self.add_output_channel)(**result)
TypeError: add_channel() got an unexpected keyword argument 'midi_channel'

I've never worked with cython before but it looks like I need to add some arguments and types to _jack_mixer.pxd and _jack_mixer.pyx so this value is recognized as a kwarg?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request to add or idea for a new feature or enhancment of an existing one
Projects
None yet
Development

No branches or pull requests

2 participants