FedeDP / Clight

A C daemon that turns your webcam into a light sensor. It will adjust screen backlight based on ambient brightness.
GNU General Public License v3.0
727 stars 27 forks source link

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

Open emansom opened 1 year ago

emansom commented 1 year ago

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 commented 1 year ago

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 :)