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.

Sign up for GitHub

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

Jump to bottom

Option to disable VU meter analysis/displays? #102

Open
mxmilkiib opened this issue Feb 8, 2021 · 7 comments · May be fixed by #151
Open

Option to disable VU meter analysis/displays? #102

mxmilkiib opened this issue Feb 8, 2021 · 7 comments · May be fixed by #151
Labels
feature request Request to add or idea for a new feature or enhancment of an existing one
Projects
Development Roadmap
Milestone
Release 17

Comments

@mxmilkiib
Copy link

I don't need to see the levels moving all the time and it's adding to the system lag I'm getting on this older laptop now I'm running a JACK/PipeWire first system.

Maybe there is a middle ground between bars and nothing that could at least denote there has not been silence on a channel in the last second?
@SpotlightKid
Copy link
Contributor

I'm currently working on disabling at least the added processing for the K14 and K20 metering, when it isn't used. I haven't measured whether this actually has a positive performance impact yet.

Adding "no meter" meter scale option should be easy to do. The backend will still update the meter values, but the impact of that is minimal.

@SpotlightKid SpotlightKid added the feature request Request to add or idea for a new feature or enhancment of an existing one label Feb 8, 2021
@SpotlightKid
Copy link
Contributor

Also, note that you can set the meter update interval in jack_mixer.py:

jack_mixer/jack_mixer.py

Line 120 in 9c4add8

GLib.timeout_add(33, self.read_meters)

It is currently set to 33 (ms). Setting it to 100 or even higher should lessen the burden on your CPU a bit, though the meters are a bit jumpy then.

@mxmilkiib
Copy link
Author

How about exposing that as a user option?

@SpotlightKid
Copy link
Contributor

SpotlightKid commented Feb 10, 2021 •
edited
Loading

Yes, that's the plan, but there are a lot of feature requests already, so I can't promise that it will happen soon.

@SpotlightKid SpotlightKid added this to High priority in Development Roadmap Feb 16, 2021
@SpotlightKid SpotlightKid added this to the Release 17 milestone Apr 9, 2021
@dsheeler
Copy link
Member

dsheeler commented Oct 2, 2021

@SpotlightKid, @mxmilkiib: what should be displayed in the "no meter" meter scale case? Fader takes up whole channel width and meter disappears altogether, or a blank meter that takes up space or ...?

@dsheeler
Copy link
Member

dsheeler commented Oct 2, 2021

Would we show the peak values?

@dsheeler dsheeler mentioned this issue Oct 2, 2021
Merged
@dsheeler
Copy link
Member

simplescreenrecorder-2021-10-25_12.26.27.mp4

@dsheeler dsheeler linked a pull request Oct 26, 2021 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request Request to add or idea for a new feature or enhancment of an existing one
Projects
Development Roadmap
High priority
Milestone
Release 17
Development

Successfully merging a pull request may close this issue.

Add button to each channel to enable/disable metering dsheeler/jack_mixer-1
3 participants
@mxmilkiib @dsheeler @SpotlightKid