davidhi7 / ddcci-plasmoid

KDE Plasma widget to adjust the brightness of multiple external monitors
MIT License
409 stars 13 forks source link

Support for setting --sleep-multiplier values #23

Open lnagel opened 1 year ago

lnagel commented 1 year ago

I would like to properly implement support for setting --sleep-multiplier 2.0 in the ddcci calls, e.g. ddcutil getvcp -d 1 -v --sleep-multiplier 2.0 10. This sleep multiplier is required to communicate with my HP z27 monitor from an AMD-based laptop.

What would be the best way to go about implementing this?

davidhi7 commented 1 year ago

I think the best solution is to add a text field to the wiget configuration that allows the user to enter a custom value as a sleep multiplier. This value is then passed to the backend, which in turn calls ddcutil with the correct sleep multiplier. I recently implemented another ddcutil specific config option this way.

Is it required to allow setting this on a per monitor basis? This would complicate it a lot.