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 I_VUMODE to the track attribute enum #55

Merged

Conversation

AlbertoV5
Copy link
Contributor

helgoboss/helgobox#560

Followed formatting on the Enum

AlbertoV5 added a commit to AlbertoV5/realearn that referenced this pull request Apr 12, 2022
helgoboss#560

and

helgoboss/reaper-rs#55

Compiled and passed the tests. Just an idea, works for me but I don't know what is best practice for mutable variables so edit as you see fit please and thank you.

Looks like all VU modes use channels 1-2, except for Multichannel and Combined RMS. In Reaper's UI the mode Stereo RMS displays different values for channel 1 and 2 but Track Peak only returns a single value so it acts like Combined Stereo RMS in practice which for me personally is fine. Then currently as of 6.53 all LUFS meters follow channels 1-2 only.
AlbertoV5 added a commit to AlbertoV5/realearn that referenced this pull request May 14, 2022
Just a heads up that TrackAttributeKey::VuMode was added here: helgoboss/reaper-rs#55
Copy link
Owner

@helgoboss helgoboss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. There's a minor thing: The doc comment is below the new enum variant. It should be above. (And the docs for the "Selected" variant should remain at their place).

@helgoboss
Copy link
Owner

I'll just merge this and fix it myself. Thanks.

@helgoboss helgoboss merged commit 5eebbaa into helgoboss:master Jun 8, 2022
@AlbertoV5
Copy link
Contributor Author

Oh sorry, I thought I had done it right. Thanks for the merge!

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.

2 participants