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

Implement Bank Select Buttons and MIDI Buttons #737

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

Conversation

diyelectromusic
Copy link
Contributor

Proposed fix for #736

Implements buttons and MIDI buttons via the following additional config items in minidexed.ini

ButtonPinBankUp=
ButtonActionBankUp=
ButtonPinBankDown=
ButtonActionBankDown=
MIDIButtonBankUp=
MIDIButtonBankDown=

Needs testing for following cases:

  • When above correspond to IO pins and PerformanceSelectChannel=0 - i.e. selecting Voice Banks
  • When above correspond to IO pins and PerformanceSelectChannel!=0 - i.e. selecting Performance Banks
  • When above are MIDI buttons and PerformanceSelectChannel=0 - i.e. selecting Voice Banks over MIDI
  • When above are MIDI buttons and PerformanceSelectChannel!=0 - i.e. selecting Performance Banks over MIDI

Note: this does not depend on the setting of PerformanceSelectToLoad - if operating on performance banks then they are always automatically loaded and selected when using buttons or MIDI buttons for selecting. Nothing else made sense to me as there is no "scroll then load" type action going on as would be the case via the UI itself.

Also, when a bank changes, the first voice/performance is automatically loaded. This might cause a bit of a delay when moving through performances maybe as I think loading performances can be a little slow...? Lets see how people get on...

So if anyone fancies giving it a go with the above configuration but also with your existing configs please do.

Kevin

Copy link

Build for testing:
MiniDexed_2024-10-21-d1da3ad
Use at your own risk.

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.

1 participant