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

Added third pitchMode #184

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

BartBral
Copy link

Added third pitchMode (2), where Start pot controls speed and Start CV controls start position.

Added third pitchMode (2), where Start pot controls speed and Start CV controls start position.
@plutoro
Copy link

plutoro commented Aug 23, 2022

also similar thing, it is possible to have this please:

the station knob to become start point and the start knob to be the pitch option (or the way around), the idea is to make a compromise and to control the same station (and change the station by just changing the bank manually) but to be able to control the start point and the pitch in the same time on the same station, and the reset to have the same function.
thanks guys ps: newby to programming in here

@dessertplanet
Copy link

In the same vein as this- in my fork I have set it up so you can toggle pitchMode from the front panel by holding the reset button until you see the LEDs flash (long-er press). The trade off is you need to normal-long-press in order to increment through the banks but that is fine for my workflow. Check it out: https://github.com/dessertplanet/RadioMusic (button-pitch-mode branch)

@dessertplanet
Copy link

I ended up wanting the behaviour described in this PR instead of my button solution- I've implemented a similar (if simpler?) version here: https://github.com/dessertplanet/RadioMusic/tree/separate-pitch-start-2 with a new boolean setting instead of a new pitch mode.

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.

3 participants