eyesome icon indicating copy to clipboard operation
eyesome copied to clipboard

Brightness value limited to 9999.99

Open heluca opened this issue 2 years ago • 1 comments

Lenovo A940 doesn't have a light sensor, so I love this app, but the max brightness value of 9999 set the display brightness to under 10%.

The '''cat /sys/class/backlight/intel_backlight/max_brightness''' output is 120000

The maximum value allowed by the eyesome-cfg.sh is 9999, so I edited lines 118 & 126 to '''MonPage+=("${CfgArr[$((i++))]}"!0.1..120000!.01!2)'''

120000 and it works as expected. I'm not sure if it's worth the effort for the config script to pull the max value automatically, but at the very least, an FYI in case someone else runs into this.

heluca avatar Oct 21 '23 12:10 heluca

Thank you for the feedback.

Initially the maximum I encountered was 1700 on one laptop. Then a newer laptop it was 7500.

With your helpful fact that 120,000 is the new maximum it makes sense to remove the limit altogether.

I'm glad you are using Eyesome and like the program.

On Sat, Oct 21, 2023 at 6:10 AM helu.ca @.***> wrote:

Lenovo A940 doesn't have a light sensor, so I love this app, but the max brightness value of 9999 set the display brightness to under 10%.

The '''cat /sys/class/backlight/intel_backlight/max_brightness''' output is 120000

The maximum value allowed by the eyesome-cfg.sh is 9999, so I edited lines 118 & 126 to '''MonPage+=("${CfgArr[$((i++))]}"!0.1..120000!.01!2)'''

120000 and it works as expected. I'm not sure if it's worth the effort for the config script to pull the max value automatically, but at the very least, an FYI in case someone else runs into this.

— Reply to this email directly, view it on GitHub https://github.com/WinEunuuchs2Unix/eyesome/issues/12, or unsubscribe https://github.com/notifications/unsubscribe-auth/AICIBH4HFVGQVTLXEQLM32LYAO3SPAVCNFSM6AAAAAA6KDHCLWVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2TKNBWGUZTGNI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

WinEunuuchs2Unix avatar Oct 21 '23 14:10 WinEunuuchs2Unix