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

Base pitch in voltSeq (note mode) #9

Closed
MarcBoule opened this issue Jan 28, 2018 · 4 comments
Closed

Base pitch in voltSeq (note mode) #9

MarcBoule opened this issue Jan 28, 2018 · 4 comments

Comments

@MarcBoule
Copy link

Hi, if I understand Rack's voltage standards correctly, could it be possible that the notes displayed in the buttons in the voltSeq sequencer, in note mode, are off by one volt? For example, when setting a button to the C4 note, the output is -1V instead of 0V. From the doc:
"If you need a default pitch for your oscillator with initial knob positions and 0V at the pitch input, use C4 (f0 = 261.626 Hz)."
https://vcvrack.com/manual/VoltageStandards.html
Not sure if my point is valid, but I get the impression all voltages output by the sequencer should be shifted up by 1V. Apologies if I misunderstood something. Thanks!

@chichian
Copy link
Collaborator

Hi Marc,

Oops. You may be correct. This is probably my fault. I didn't think there was a standard other than C's were even voltages, so I went -5 to +5 V (symmetry is always nice and I know nothing about music).

Let us look into this and how I can change it and hopefully gracefully deal with previous saves with a conversion to -4 to +6V.

@MarcBoule
Copy link
Author

Thanks for the followup. If ever there is no graceful way to make the change, perhaps waiting until the change to Rack v.0.6 would be best, to synchronize with that more major event. Cheers!

@chichian
Copy link
Collaborator

Hi Marc,

I've changed it in this 0.5.5.2 release. The 'notes' should be the same as what was displayed to the user before, but the voltages will be +1 more. I've added a "Shift" voltages in the context menu to compensate for this (in case the users actually wanted the old voltages).

Thanks for catching that!

@MarcBoule
Copy link
Author

Awesome! The new options are actually a cool new feature also, and would find use in other cases as well I imagine. Thanks for the fix, and your sequencers are superb by the way! 👍

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

No branches or pull requests

2 participants