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

[QUESTION/FEATURE REQ] glob matching for override monitor_id #280

Open
emansom opened this issue Jun 30, 2023 · 1 comment
Open

[QUESTION/FEATURE REQ] glob matching for override monitor_id #280

emansom opened this issue Jun 30, 2023 · 1 comment

Comments

@emansom
Copy link

emansom commented Jun 30, 2023

Is your feature request related to a problem? Please describe.
I would like to limit the max brightness of a certain backlight capable monitor to 60% at max to extend its useful life. For this, I'd like to use the monitor_override options.

However, the DDC/CI exposed backlight (through the out-of-tree kernel module) sometimes makes the monitor appear with a different identifier (e.g. ddcci5 instead of ddcci0).

This results in the override no longer functioning, and the max brightness going back to 100% again.

Afaict there's currently no way to target all monitors that start with ddcci in the identifier?

Describe the solution you'd like
The monitor_override config to support targeting N monitors through glob matching in the monitor_id field.

Describe alternatives you've considered
I'm aware of the sensor point overrides functionality, however that seems like the wrong place to place a limiter; as it's not the sensor data that needs altering.

@FedeDP
Copy link
Owner

FedeDP commented Aug 17, 2023

Hi! Sorry, I completely lost track of this feat request!
I think it makes sense, thanks for the input!
I will try to give this a shoot for the next release :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants