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

Volume matching #381

Merged
merged 3 commits into from
Aug 19, 2020
Merged

Volume matching #381

merged 3 commits into from
Aug 19, 2020

Conversation

jpcima
Copy link
Collaborator

@jpcima jpcima commented Aug 19, 2020

This matches volumes to ARIA exactly. Fixes #367
It's tried with waveforms, mono samples, stereo samples.

It does so by connecting CC7 to region amplitude, and initializing the CC to 100, with concave curve.
The CC10 will be connected with pan, with default to middle, with bipolar curve.

The stereo samples needed +3dB, because the succession of pan and position attenuate by -6dB in total.

@jpcima jpcima requested a review from paulfd August 19, 2020 05:19
@jpcima jpcima merged commit 95165ca into sfztools:develop Aug 19, 2020
@kinwie
Copy link

kinwie commented Aug 19, 2020

Yes, that will be much convenient.
If i may add another suggest, set the CC value to 96, not 100. I'm pretty sure ARIA use CC7 value 96, and also this is the default value in most DAW midi track afaik. For example, if my midi track volume has been modified, I just "Alt+click" its fader to set it to default and it will go to value 96. It is match to ARIA's -4 dB default (or exactly -4.188 dB).

@jpcima
Copy link
Collaborator Author

jpcima commented Aug 19, 2020

Where does this value 96 come from?
MIDI 96 is -4.86 dB

@paulfd
Copy link
Member

paulfd commented Aug 19, 2020

96 is 3/4 of 128 maybe?

@kinwie
Copy link

kinwie commented Aug 19, 2020

Ahh, sorry, just got home and re-check this.
@jpcima you are right. It should be midi cc value 100 to match ARIA's -4 dB.
My bad memory...getting old :)

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.

Request to reduce main volume to -3 or -4 or -6 dB
3 participants