FedeDP / Clightd

A linux bus interface that lets you change screen brightness, compute captured webcam frames brightness and change screen temperature.
GNU General Public License v3.0
82 stars 10 forks source link

Doesn't work on Asus UW5606 despite accurate sensor readings #117

Open ThatOneCalculator opened 1 month ago

ThatOneCalculator commented 1 month ago

For some background, this is on the Asus UM5606 (ZenBook S 16 2024). The brightness adjustment via the ambient light sensor on GNOME works perfectly. However, I looked into using clight as I want to use a standalone wayland compositor. I'm also able to adjust the brightness of the backlight manually. Furthermore, the output of busctl call org.clightd.clightd /org/clightd/clightd/Sensor/Als org.clightd.clightd.Sensor Capture "sis" "" 5 "" gives accurate readings. However, brightness does not adjust at all when using clight.

Going through the wizard, here's my output:

clight -w
Welcome to Clight wizard. Press ctrl-c to quit at any moment.
Wizard will use 'amdgpu_bl1' screen.
Waiting for sensor...
Sensor available. Start? [Y/n]: >

Move to nightlight-like light.
Are you ready? [Y/n]

Ambient brightness is currently 0.000; redo capture? [y/N]: >
Ambient brightness is currently 0.000; redo capture? [y/N]: > n
Set desired screen backlight then press ENTER... > 5

Backlight level is: 1.000

Move to a dark room.
Are you ready? [Y/n]

Ambient brightness is currently 0.000; redo capture? [y/N]: > y
Ambient brightness is currently 0.000; redo capture? [y/N]: > y
Ambient brightness is currently 0.000; redo capture? [y/N]: > y
Ambient brightness is currently 0.000; redo capture? [y/N]: >
Ambient brightness is currently 0.000; redo capture? [y/N]: > y
Ambient brightness is currently 0.065; redo capture? [y/N]: > n
Set desired screen backlight then press ENTER... > 50

Backlight level is: 1.000

Move to normal-light room.
Are you ready? [Y/n]

Ambient brightness is currently 0.196; redo capture? [y/N]: > y
Ambient brightness is currently 0.073; redo capture? [y/N]: > y
Ambient brightness is currently 0.072; redo capture? [y/N]: > n
Set desired screen backlight then press ENTER... > 60

Backlight level is: 1.000

Move to a bright room.
Are you ready? [Y/n]

Ambient brightness is currently 0.211; redo capture? [y/N]: > y
Ambient brightness is currently 0.213; redo capture? [y/N]: > n
Set desired screen backlight then press ENTER... > 75

Backlight level is: 1.000

Move to daylight-like light.
Are you ready? [Y/n]

Ambient brightness is currently 0.254; redo capture? [y/N]: > y
Ambient brightness is currently 0.258; redo capture? [y/N]: > n
Set desired screen backlight then press ENTER... > 100

Backlight level is: 1.000

Computing new backlight curve...
Computing new regression points...
[ 1.000,  1.000,  1.000,  1.000,  1.000,  1.000,  1.000,  1.000,  1.000,  1.000,  1.000  ]
Don't forget to set these points in clight conf file!
Bye!

Which is a bit strange, since it gets the ambient brightness level but constantly sets the backlight to "1.0000".

Running clight with no args just keeps the brightness at max too, and using clight-gui it seems that it's not getting any readings from the ambient light sensor.

ThatOneCalculator commented 1 month ago

Going through the wizard again, it seems to have the most problems with setting the backlight. I did this in an entirely dark room, so the reading of 0 is accurate, but it seems to want to make my brightness 100%. I started the test at 80% (manually set)

image