search

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
83 stars 10 forks source link

[BUG] Not setting backlight on latest release #69

Closed hunterzero99 closed 3 years ago

hunterzero99 commented 3 years ago

Clight version the issue has been seen with:

latest release and git (clight & clightd from AUR, also clight-git and clightd-git from AUR

Describe the bug Backlight level is not being changed or reported correctly for i2c devices. I've tried setting i2c device directly, clight always reports 0.00

Backlight 'J0DKG7120D3L' level updated: 0.00

The correct backlight level is being reported from gddccontrol, ddcui and can be changed with both of those tools. It used to work on this hardware ~2 to 3 weeks ago.

Verbose log:

`Clight
* Software version:     4.5
* Global config dir:        /etc/default
* Global data dir:      /usr/share/clight
* Starting time:        Thu Mar 11 07:31:14 2021

Starting options:

### GENERIC ###
* Verbose (debug):      Enabled
* ResumeDelay:      0

### BACKLIGHT ###
* Smooth trans:     Enabled
* Smooth steps:     0.05
* Smooth timeout:       30
* Daily timeouts:       AC 10   BATT 10
* Nightly timeout:      AC 10   BATT 10
* Event timeouts:       AC 10   BATT 10
* Backlight path:       Unset
* Shutter threshold:        0.00
* Autocalibration:      Disabled
* Pause on lid closed:      Disabled
* Capture on lid opened:        Disabled

### SENSOR ###
* Captures:     AC 2    BATT 2
* Device:       /dev/bus/usb/005/002
* Settings:     Unset

### GAMMA ###
* Smooth trans:     Enabled
* Smooth steps:     50
* Smooth timeout:       300
* Daily screen temp:        6500
* Nightly screen temp:      4000
* Long transition:      Enabled
* Ambient gamma:        Disabled

### DAYTIME ###
* User position:        41.89   -71.41
* User set sunrise:     Unset
* User set sunset:      Unset
* Event duration:       1800

### DIMMER ###
* Smooth trans:     ENTER: Enabled, EXIT: Enabled
* Smooth steps:     ENTER: 0.05, EXIT: 0.05
* Smooth timeout:       ENTER: 30, EXIT: 30
* Timeouts:     AC 45   BATT 20
* Backlight pct:        0.20

### DPMS ###
* Timeouts:     AC 900  BATT 300

### SCREEN ###
* Timeouts:     AC -1   BATT -1
* Contrib:      0.10
* Samples:      10

### INHIBIT ###
* Docked:       Disabled
* PowerManagement:      Disabled

(I)[07:31:14]{main.c:127}   Clightd found, version: 5.3.
(I)[07:31:14]{backlight.c:418}  AC curve: y = -0.025538 + 0.182420x + -0.007951x^2
BL
^
|        **
|      **  
|     *    
|    *     
|   *      
|          
|  *       
| *        
|          
|*         
*---------->BR
(I)[07:31:14]{backlight.c:418}  BATT curve: y = -0.025538 + 0.182420x + -0.007951x^2
BL
^
|        **
|      **  
|     *    
|    *     
|   *      
|          
|  *       
| *        
|          
|*         
*---------->BR
(W)[07:31:14]{interface.c:402}  Failed to create org.freedesktop.ScreenSaver dbus interface: Unknown error -1
(I)[07:31:14]{interface.c:403}  Fallback at monitoring requests to org.freedesktop.ScreenSaver name owner.
(D)[07:31:14]{timer.c:34}   Disarmed timerfd on fd 36.
(I)[07:31:14]{upower.c:95}  Not a laptop device. Killing UPower module.
(W)[07:31:14]{upower.c:45}  Failed to retrieve AC state; fallback to ON_AC and OPEN lid.
(D)[07:31:14]{interface.c:455}  Emitting 'AcState' property
(D)[07:31:14]{timer.c:34}   Disarmed timerfd on fd 39.
(D)[07:31:14]{interface.c:455}  Emitting 'LidState' property
(D)[07:31:14]{timer.c:32}   Set timeout of 0s 1ns on fd 40.
(I)[07:31:14]{daytime.c:175}    Next alarm due to: Thu Mar 11 17:15:00 2021
(D)[07:31:14]{timer.c:32}   Set timeout of 35026s 0ns on fd 40.
(D)[07:31:14]{interface.c:455}  Emitting 'Sunrise' property
(D)[07:31:14]{interface.c:455}  Emitting 'Sunset' property
(D)[07:31:14]{interface.c:455}  Emitting 'DayTime' property
(I)[07:31:14]{gamma.c:186}  Normal transition to 6500 gamma temp.
(D)[07:31:14]{interface.c:455}  Emitting 'NextEvent' property
(D)[07:31:14]{timer.c:32}   Set timeout of 0s 1ns on fd 41.
(D)[07:31:14]{backlight.c:291}  Sensor '/dev/bus/usb/005/002' is now available.
(D)[07:31:14]{backlight.c:497}  Resumed as a sensor is now available.
(D)[07:31:14]{utils.c:56}   Pausing BACKLIGHT
(D)[07:31:14]{interface.c:455}  Emitting 'Temp' property
(D)[07:31:14]{interface.c:455}  Emitting 'Temp' property
(D)[07:31:14]{backlight.c:514}  Backlight 'J0DKG7120D3L' level updated: 0.00.
(D)[07:31:14]{interface.c:455}  Emitting 'SensorAvail' property
(D)[07:31:14]{interface.c:455}  Emitting 'BlPct' property
(D)[07:31:14]{interface.c:455}  Emitting 'BlPct' property`
FedeDP commented 3 years ago

Hi! Thanks for opening this bug report! Most probably this is a bug due to some Clightd changes; i wil transfer this bug to Clightd repo!

In the meantime, can you share journalctl -bu clightd output?

hunterzero99 commented 3 years ago 
``-- Journal begins at Sun 2021-03-07 10:33:51 EST, ends at Thu 2021-03-11 12:18:03 EST. --
Mar 11 09:09:09 acamar systemd[1]: Starting Bus service to manage various screen related properties (gamma, dpms, backlight)...
Mar 11 09:09:09 acamar systemd[1]: Started Bus service to manage various screen related properties (gamma, dpms, backlight).
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Als' sensor plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Camera' sensor plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Custom' sensor plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'YoctoLight' sensor plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Drm' gamma plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Wl' gamma plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Xorg' gamma plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Drm' dpms plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Wl' dpms plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Xorg' dpms plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Fb' screen plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Wl' screen plugin.
Mar 11 09:09:10 acamar clightd[1159]: Registered 'Xorg' screen plugin.
Mar 11 09:09:10 acamar clightd[1159]: [default]|IDLE|: Creating client 0
Mar 11 09:09:10 acamar clightd[1159]: [default]|IDLE|: Adding inotify watch as first client was started.
Mar 11 09:09:10 acamar clightd[1159]: [default]|IDLE|: Starting Client 0
Mar 11 09:09:10 acamar clightd[1159]: [default]|IDLE|: Creating client 1
Mar 11 09:09:10 acamar clightd[1159]: [default]|IDLE|: Starting Client 1
Mar 11 09:09:10 acamar clightd[1159]: [default]|GAMMA|: Temperature target value set: 6500.
Mar 11 09:09:10 acamar clightd[1159]: [default]|GAMMA|: Reached target temp: 6500.
Mar 11 10:29:44 acamar clightd[1159]: [default]|BACKLIGHT|: Target pct: 1.00
Mar 11 10:29:44 acamar clightd[1159]: [default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L
Mar 11 10:29:44 acamar clightd[1159]: [default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 1.00.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Next timer: 43
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Stopping Client 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Removing inotify watch as only client using it was stopped.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Stopping Client 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Adding inotify watch as first client was started.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Starting Client 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Starting Client 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Stopping Client 1
Mar 11 10:29:44 acamar clightd[1159]: [default]|IDLE|: Removing inotify watch as only client using it was stopped.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Stopping Client 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Adding inotify watch as first client was started.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Starting Client 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Starting Client 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Stopping Client 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Removing inotify watch as only client using it was stopped.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Stopping Client 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Adding inotify watch as first client was started.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Starting Client 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Starting Client 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Stopping Client 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Removing inotify watch as only client using it was stopped.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Stopping Client 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Adding inotify watch as first client was started.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Starting Client 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Starting Client 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 1
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Leaving idle state.
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 1 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Client 0 -> Idle: 0
Mar 11 11:52:27 acamar clightd[1159]: [default]|IDLE|: Stopping Client 1
FedeDP commented 3 years ago

Thank you!

Mar 11 10:29:44 acamar clightd[1159]: [default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L Mar 11 10:29:44 acamar clightd[1159]: [default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 1.00.

As you can see, it fails to set backlight. To avoid a deadend loop, it ends the "smooth" transition.

The weird thing is that i only see a single request to set backlight. Can you double check with: busctl call org.clightd.clightd /org/clightd/clightd/Backlight org.clightd.clightd.Backlight SetAll "d(bdu)s" 0.5 0 0 0 "" and see if this changes your screen backlight?

I noticed that sometimes when resuming from sleep or dpms state, Clight is too quick to ask the monitor for a new backlight, receiving above error.

hunterzero99 commented 3 years ago

That command outputs 'b true' with no change to my backlight.

I don't have suspend/resume or sleep mode one this machine at all. dpms rarely runs on this monitor either. I think I have a very uncommon monitor, though.

On Thu, Mar 11, 2021 at 12:56 PM Federico Di Pierro < @.***> wrote:

Thank you!

Mar 11 10:29:44 acamar clightd[1159]: [default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L Mar 11 10:29:44 acamar clightd[1159]: [default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 1.00.

As you can see, it fails to set backlight. To avoid a deadend loop, it ends the "smooth" transition.

The weird thing is that i only see a single request to set backlight. Can you double check with: busctl call org.clightd.clightd /org/clightd/clightd/Backlight org.clightd.clightd.Backlight SetAll "d(bdu)s" 0.5 0 0 0 "" and see if this changes your screen backlight?

I noticed that sometimes when resuming from sleep or dpms state, Clight is too quick to ask the monitor for a new backlight, receiving above error.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FedeDP/Clightd/issues/69#issuecomment-796925995, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJROONW2QQSXZ42GD6IS3UDTDD73BANCNFSM4ZAUL3XQ .

FedeDP commented 3 years ago

Ok, thank you for the test. I will create a new branch adding some debug statements, are you willing to help me debug this? I will need you to:

It is much harder to write it than to do it :) Thank you very much!

hunterzero99 commented 3 years ago

Absolutely I'm up to help out in any way I can! Thank you for your efforts.

On Thu, Mar 11, 2021 at 1:22 PM Federico Di Pierro @.***> wrote:

Ok, thank you for the test. I will create a new branch adding some debug statements, are you willing to help me debug this? I will need you to:

  • build clightd from the new branch with ddcutil enabled (cmake -DENABLE_DDC=1 ../)
  • kill Clight: killall clight
  • stop currently running Clightd: sudo systemctl stop clightd
  • finally start built Clightd: sudo ./clightd
  • call above busctl command
  • paste here output

It is much harder to write it than to do it :) Thank you very much!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FedeDP/Clightd/issues/69#issuecomment-796944749, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJROONSZVFEH2PFE5VLOZXDTDEC4RANCNFSM4ZAUL3XQ .

FedeDP commented 3 years ago

Great! Just pushed DDC_fixes branch! Let me know!

hunterzero99 commented 3 years ago

sudo ./clightd Registered 'Als' sensor plugin. Registered 'Camera' sensor plugin. Registered 'Custom' sensor plugin. [default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor [default]|BACKLIGHT|: SET 0x5618742d6e00 monitor ref; max monitor bl: 100, curr: 50 [default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor [default]|BACKLIGHT|: Target pct: 1.00 [default]|BACKLIGHT|: Opening (nil) monitor handle [default]|BACKLIGHT|: End with: -1 [default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L [default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 1.00.

busctl call org.clightd.clightd /org/clightd/clightd/Backlight org.clightd.clightd.Backlight SetAll "d(bdu)s" 1.0 0 0 0 "" b true

On Thu, Mar 11, 2021 at 1:25 PM Federico Di Pierro @.***> wrote:

Great! Just pushed DDC_fixes branch! Let me know!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FedeDP/Clightd/issues/69#issuecomment-796947245, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJROONQXZ7OUPFIAHDCXWFLTDEDK7ANCNFSM4ZAUL3XQ .

FedeDP commented 3 years ago

Interesting! I did another small push, can you test again? Thanks!

hunterzero99 commented 3 years ago

Same output as before. I tried changing it a couple of time 

sudo ./clightd 
Registered 'Als' sensor plugin.
Registered 'Camera' sensor plugin.
Registered 'Custom' sensor plugin.
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor
[default]|BACKLIGHT|: SET 0x56464ba1be00 monitor ref; max monitor bl: 100, curr: 50
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor
[default]|BACKLIGHT|: Target pct: 1.00
[default]|BACKLIGHT|: Opening (nil) monitor ('J0DKG7120D3L') handle, max br: -1
[default]|BACKLIGHT|: End with: -1
[default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 1.00.
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor
[default]|BACKLIGHT|: SET 0x56464ba1be00 monitor ref; max monitor bl: 100, curr: 50
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor
[default]|BACKLIGHT|: Target pct: 0.50
[default]|BACKLIGHT|: Opening (nil) monitor ('J0DKG7120D3L') handle, max br: -1
[default]|BACKLIGHT|: End with: -1
[default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 0.50.
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor
[default]|BACKLIGHT|: SET 0x56464ba1be00 monitor ref; max monitor bl: 100, curr: 50
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor
[default]|BACKLIGHT|: Target pct: 0.25
[default]|BACKLIGHT|: Opening (nil) monitor ('J0DKG7120D3L') handle, max br: -1
[default]|BACKLIGHT|: End with: -1
[default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 0.25.
FedeDP commented 3 years ago

It has a very nice information though :)

[default]|BACKLIGHT|: Opening (nil) monitor ('J0DKG7120D3L') handle, max br: -1

It seems like something is resetting the structure. And here is that "something":

[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor [default]|BACKLIGHT|: SET 0x56464ba1be00 monitor ref; max monitor bl: 100, curr: 50 [default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor

It seems like it adds twice the same monitor and somewhat weird things happen! I added a new debug log, can you test again?

hunterzero99 commented 3 years ago 
sudo ./build/clightd 
Registered 'Als' sensor plugin.
Registered 'Camera' sensor plugin.
Registered 'Custom' sensor plugin.
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: 0x55ec271c5b50
[default]|BACKLIGHT|: SET 0x55ec271c9e00 monitor ref on 0x55ec271c5b50 sc; max monitor bl: 100, curr: 50
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: (nil)
[default]|BACKLIGHT|: Target pct: 0.60
[default]|BACKLIGHT|: Opening (nil) monitor ('J0DKG7120D3L') handle, max br: -1
[default]|BACKLIGHT|: End with: -1
[default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 0.60.
FedeDP commented 3 years ago

While there is a quick fix for your issue, i just added a couple of more debug logs to help me understand what's causing it. I mean, the fix is trivial, but i don't understand why it happens and i'd like to go deep inside it! Can you pull and re-test? Thanks! Last time hopefully! :D

hunterzero99 commented 3 years ago

I think I may have figured out the why. I actually have a monitor with multiple inputs, and I have this set up receive an input from one DisplayPort and one HDMI connection. I've unplugged one and then brightness changing started to work again. If both are plugged in, then it does not work. The thing is it was working a few weeks ago with my setup.

In the log below I unplugged both then successfully change the brightness.

sudo ./build/clightd 
Registered 'Als' sensor plugin.
Registered 'Camera' sensor plugin.
Registered 'Custom' sensor plugin.
[default]|BACKLIGHT|: Adding 'J0DKG7120D3L' monitor
[default]|BACKLIGHT|: Put 'J0DKG7120D3L' monitor (0x557d212e7710): 0
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: 0x557d212e7710
[default]|BACKLIGHT|: SET 0x557d212fce00 monitor ref on 0x557d212e7710 sc; max monitor bl: 100, curr: 30
[default]|BACKLIGHT|: Adding 'J0DKG7120D3L' monitor
[default]|BACKLIGHT|: Put 'J0DKG7120D3L' monitor (0x557d212f35e0): 0
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: (nil)
[default]|BACKLIGHT|: Target pct: 0.50
[default]|BACKLIGHT|: Opening (nil) monitor ('J0DKG7120D3L') handle, max br: -1
[default]|BACKLIGHT|: End with: -1
[default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 0.50.
(check_displays                ) Displays changed!
(check_displays                ) Previous connected displays: card0-DP-1, card0-HDMI-A-1
(check_displays                ) Current  connected displays: card0-DP-1
[default]|BACKLIGHT|: Adding 'J0DKG7120D3L' monitor
[default]|BACKLIGHT|: Put 'J0DKG7120D3L' monitor (0x557d212e7850): 0
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: 0x557d212e7850
[default]|BACKLIGHT|: SET 0x557d212ffa90 monitor ref on 0x557d212e7850 sc; max monitor bl: 100, curr: 30
[default]|BACKLIGHT|: Target pct: 0.50
[default]|BACKLIGHT|: Opening 0x557d212ffa90 monitor ('J0DKG7120D3L') handle, max br: 100
[default]|BACKLIGHT|: Opened 0x557d212ffa90 monitor handle
[default]|BACKLIGHT|: Set bl: 0
[default]|BACKLIGHT|: End with: 0
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 0.50.
(check_displays                ) Displays changed!
(check_displays                ) Previous connected displays: card0-DP-1
(check_displays                ) Current  connected displays: card0-DP-1, card0-HDMI-A-1
(check_displays                ) Displays changed!
(check_displays                ) Previous connected displays: card0-DP-1, card0-HDMI-A-1
(check_displays                ) Current  connected displays: card0-HDMI-A-1
[default]|BACKLIGHT|: Adding 'J0DKG7120D3L' monitor
[default]|BACKLIGHT|: Put 'J0DKG7120D3L' monitor (0x557d212e74e0): 0
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: 0x557d212e74e0
[default]|BACKLIGHT|: SET 0x557d212fce00 monitor ref on 0x557d212e74e0 sc; max monitor bl: 100, curr: 50
[default]|BACKLIGHT|: Target pct: 0.50
[default]|BACKLIGHT|: Opening 0x557d212fce00 monitor ('J0DKG7120D3L') handle, max br: 100
[default]|BACKLIGHT|: Opened 0x557d212fce00 monitor handle
[default]|BACKLIGHT|: Set bl: 0
[default]|BACKLIGHT|: End with: 0
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 0.50.
[default]|BACKLIGHT|: Adding 'J0DKG7120D3L' monitor
[default]|BACKLIGHT|: Put 'J0DKG7120D3L' monitor (0x557d212f7870): 0
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: 0x557d212f7870
[default]|BACKLIGHT|: SET 0x557d212fce00 monitor ref on 0x557d212f7870 sc; max monitor bl: 100, curr: 50
[default]|BACKLIGHT|: Target pct: 0.30
[default]|BACKLIGHT|: Opening 0x557d212fce00 monitor ('J0DKG7120D3L') handle, max br: 100
[default]|BACKLIGHT|: Opened 0x557d212fce00 monitor handle
[default]|BACKLIGHT|: Set bl: 0
[default]|BACKLIGHT|: End with: 0
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 0.30.
FedeDP commented 3 years ago

Yep, can you give me an old output with both DisplayPort and HDMI connections? I just want to better understand what's happening, then i'll fix the issue :)

hunterzero99 commented 3 years ago

I'm doing the same thing, pulling the latest commit from DDC_fixes but now I'm getting next to no output. I feel like I'm going something obviously wrong here...

Either way, thank you for all your fixes!

sudo ./build/clightd --verbose Registered 'Als' sensor plugin. Registered 'Camera' sensor plugin. Registered 'Custom' sensor plugin. [default]|BACKLIGHT|: Target pct: 0.50 [default]|BACKLIGHT|: Target pct: 1.00

hunterzero99 commented 3 years ago

Here's the output you want, sorry about that. For some reason it started working after I launched gddcutil to test.

sudo ./build/clightd
Registered 'Als' sensor plugin.
Registered 'Camera' sensor plugin.
Registered 'Custom' sensor plugin.
[default]|BACKLIGHT|: Adding 'J0DKG7120D3L' monitor
[default]|BACKLIGHT|: Put 'J0DKG7120D3L' monitor (0x561ec6a09600): 0
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: 0x561ec6a09600
[default]|BACKLIGHT|: SET 0x561ec6a13e00 monitor ref on 0x561ec6a09600 sc; max monitor bl: 100, curr: 63
[default]|BACKLIGHT|: Adding 'J0DKG7120D3L' monitor
[default]|BACKLIGHT|: Put 'J0DKG7120D3L' monitor (0x561ec69ff280): 0
[default]|BACKLIGHT|: ADDED J0DKG7120D3L monitor. Found 2 monitors. Sc: (nil)
[default]|BACKLIGHT|: Target pct: 1.00
[default]|BACKLIGHT|: Opening (nil) monitor ('J0DKG7120D3L') handle, max br: -1
[default]|BACKLIGHT|: End with: -1
[default]|BACKLIGHT|: failed to set backlight for J0DKG7120D3L
[default]|BACKLIGHT|: J0DKG7120D3L reached target backlight: 1.00.
FedeDP commented 3 years ago

No problem sir! Ok i pushed a commit on master to fix your issue! Let me know! Thank you for all the debugging!

hunterzero99 commented 3 years ago

Changing the brightness via dbus is working now, thank you very much! Oddly I'm still not getting the brightness to change automatically, though. I'm getting data from the sensor via dbus, and it lets me run the wizard. I specified the device manually to make sure it wasn't an error there (Yocto).

busctl call org.clightd.clightd /org/clightd/clightd/Sensor org.clightd.clightd.Sensor Capture "sis" "" 5 ""
sad "/dev/bus/usb/005/005" 5 0.0976 0.0976 0.0976 0.0976 0.0976

output of .local/share/clight/clight.log:

`(W)[17:34:35]{config.c:310}    Config file /home/crdaley/.config/clight.conf not found.

Clight
* Software version:     4.5
* Global config dir:        /etc/default
* Global data dir:      /usr/share/clight
* Starting time:        Thu Mar 11 17:34:35 2021

Starting options:

### GENERIC ###
* Verbose (debug):      Enabled
* ResumeDelay:      0

### BACKLIGHT ###
* Smooth trans:     Enabled
* Smooth steps:     0.05
* Smooth timeout:       30
* Daily timeouts:       AC 10   BATT 10
* Nightly timeout:      AC 10   BATT 10
* Event timeouts:       AC 10   BATT 10
* Backlight path:       Unset
* Shutter threshold:        0.00
* Autocalibration:      Enabled
* Pause on lid closed:      Disabled
* Capture on lid opened:        Disabled

### SENSOR ###
* Captures:     AC 5    BATT 5
* Device:       /dev/bus/usb/005/004
* Settings:     Unset

### KEYBOARD ###
* Dim:      Disabled
* Threshold:        1.00
* Timeouts:     AC 15   BATT 5

### GAMMA ###
* Smooth trans:     Enabled
* Smooth steps:     50
* Smooth timeout:       300
* Daily screen temp:        6500
* Nightly screen temp:      4000
* Long transition:      Disabled
* Ambient gamma:        Disabled

### DAYTIME ###
* User position:        Unset
* User set sunrise:     Unset
* User set sunset:      Unset
* Event duration:       1800

### DIMMER ###
* Smooth trans:     ENTER: Enabled, EXIT: Enabled
* Smooth steps:     ENTER: 0.05, EXIT: 0.05
* Smooth timeout:       ENTER: 30, EXIT: 30
* Timeouts:     AC 45   BATT 20
* Backlight pct:        0.20

### DPMS ###
* Timeouts:     AC 900  BATT 300

### SCREEN ###
* Timeouts:     AC 30   BATT -1
* Contrib:      0.10
* Samples:      10

### INHIBIT ###
* Docked:       Disabled
* PowerManagement:      Disabled

(I)[17:34:35]{main.c:127}   Clightd found, version: 5.3.
(I)[17:34:35]{backlight.c:418}  AC curve: y = -0.025538 + 0.182420x + -0.007951x^2
BL
^
|        **
|      **  
|     *    
|    *     
|   *      
|          
|  *       
| *        
|          
|*         
*---------->BR
(I)[17:34:35]{backlight.c:418}  BATT curve: y = -0.025538 + 0.182420x + -0.007951x^2
BL
^
|        **
|      **  
|     *    
|    *     
|   *      
|          
|  *       
| *        
|          
|*         
*---------->BR
(D)[17:34:35]{bus.c:181}    init_kbd_backlight(): No such file or directory
(I)[17:34:35]{keyboard.c:129}   Keyboard backlight calibration unsupported.
(W)[17:34:35]{interface.c:402}  Failed to create org.freedesktop.ScreenSaver dbus interface: Unknown error -1
(I)[17:34:35]{interface.c:403}  Fallback at monitoring requests to org.freedesktop.ScreenSaver name owner.
(D)[17:34:35]{timer.c:34}   Disarmed timerfd on fd 40.
(I)[17:34:35]{upower.c:95}  Not a laptop device. Killing UPower module.
(W)[17:34:35]{upower.c:45}  Failed to retrieve AC state; fallback to ON_AC and OPEN lid.
(D)[17:34:35]{interface.c:455}  Emitting 'AcState' property
(D)[17:34:35]{timer.c:32}   Set timeout of 0s 1ns on fd 43.
(D)[17:34:35]{interface.c:455}  Emitting 'LidState' property
(D)[17:34:35]{timer.c:32}   Set timeout of 30s 0ns on fd 43.
(W)[17:34:35]{location.c:101}   Error loading from cache file.
(D)[17:35:05]{timer.c:32}   Set timeout of 30s 0ns on fd 43.
(I)[17:35:06]{signal.c:48}  Received 2. `Leaving.``
FedeDP commented 3 years ago

Changing the brightness via dbus is working now, thank you very much!

Great to hear!

Oddly I'm still not getting the brightness to change automatically, though. I'm getting data from the sensor via dbus, and it lets me run the wizard.

Did you install new clightd? After installing it you should stop old one, run daemon-reload and restart new one, ie: sudo systemctl stop clightd && sudo systemctl daemon-reload && sudo systemctl start clightd Normally, i just reboot; that way i am sure it is actually upgraded :)

I specified the device manually to make sure it wasn't an error there (Yocto).

Great to hear you are also using YoctoLight sensor! I do not use mine often (on laptop devices it is quite annoying to carry the thing connected to usb :D ) Anyway, it is a nice device!

hunterzero99 commented 3 years ago

Yes, I did stop, reload and restart the service. I even tried completely wiping out all the config files and rebooting.

On Thu, Mar 11, 2021 at 5:49 PM Federico Di Pierro @.***> wrote:

Changing the brightness via dbus is working now, thank you very much!

Great to hear!

Oddly I'm still not getting the brightness to change automatically, though. I'm getting data from the sensor via dbus, and it lets me run the wizard.

Did you install new clightd? After installing it you should stop old one, run daemon-reload and restart new one, ie: sudo systemctl stop clightd && sudo systemctl daemon-reload && sudo systemctl start clightd

I specified the device manually to make sure it wasn't an error there (Yocto).

Great to hear you are also using YoctoLight sensor! I do not use mine often (on laptop devices it is quite annoying to carry the thing connected to usb :D ) Anyway, it is a nice device!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/FedeDP/Clightd/issues/69#issuecomment-797101773, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJROONQ6LKHZB2OLF4DMCY3TDFCH5ANCNFSM4ZAUL3XQ .

FedeDP commented 3 years ago

Yes, I did stop, reload and restart the service.

So, if you now run again the busctl command to change backlight, it works uh?

hunterzero99 commented 3 years ago

So, if you now run again the busctl command to change backlight, it works uh?

Yes, it does, thank you!

Now I'm getting these errors in the log, and the backlight is not changing: capture_frames_brightness(): Input/output error

I have run through the Polkit Support section of the wiki and set up /etc/polkit-1/rules.d/80-clightd.rules

and this command dose output the sensor's values: busctl call org.clightd.clightd /org/clightd/clightd/Sensor/Camera org.clightd.clightd.Sensor Capture "sis" "" 5 "" sad "/dev/video0" 5 0.156863 0.156863 0.156863 0.156863 0.156863

Verbose log output:

(D)[06:54:26]{opts.c:442}   Inhibit module is not needed. Disabling.

Clight
* Software version:     4.5
* Global config dir:        /etc/default
* Global data dir:      /usr/share/clight
* Starting time:        Fri Mar 12 06:54:26 2021

Starting options:

### GENERIC ###
* Verbose (debug):      Enabled
* ResumeDelay:      0

### BACKLIGHT ###
* Smooth trans:     Enabled
* Smooth steps:     0.05
* Smooth timeout:       30
* Daily timeouts:       AC 10   BATT 10
* Nightly timeout:      AC 10   BATT 10
* Event timeouts:       AC 10   BATT 10
* Backlight path:       Unset
* Shutter threshold:        0.00
* Autocalibration:      Enabled
* Pause on lid closed:      Disabled
* Capture on lid opened:        Disabled

### SENSOR ###
* Captures:     AC 5    BATT 5
* Device:       /dev/bus/usb/005/002
* Settings:     Unset

### DAYTIME ###
* User position:        41.89   -71.41
* User set sunrise:     Unset
* User set sunset:      Unset
* Event duration:       1800

(I)[06:54:26]{main.c:127}   Clightd found, version: 5.3.
(I)[06:54:26]{backlight.c:418}  AC curve: y = -0.025538 + 0.182420x + -0.007951x^2
BL
^
|        **
|      **  
|     *    
|    *     
|   *      
|          
|  *       
| *        
|          
|*         
*---------->BR
(I)[06:54:26]{backlight.c:418}  BATT curve: y = -0.025538 + 0.182420x + -0.007951x^2
BL
^
|        **
|      **  
|     *    
|    *     
|   *      
|          
|  *       
| *        
|          
|*         
*---------->BR
(I)[06:54:26]{upower.c:95}  Not a laptop device. Killing UPower module.
(W)[06:54:26]{upower.c:45}  Failed to retrieve AC state; fallback to ON_AC and OPEN lid.
(D)[06:54:26]{interface.c:455}  Emitting 'AcState' property
(D)[06:54:26]{timer.c:32}   Set timeout of 0s 1ns on fd 22.
(D)[06:54:26]{interface.c:455}  Emitting 'LidState' property
(I)[06:54:26]{daytime.c:175}    Next alarm due to: Fri Mar 12 17:16:00 2021
(D)[06:54:26]{timer.c:32}   Set timeout of 37294s 0ns on fd 22.
(D)[06:54:26]{interface.c:455}  Emitting 'Sunrise' property
(D)[06:54:26]{interface.c:455}  Emitting 'Sunset' property
(D)[06:54:26]{timer.c:32}   Set timeout of 0s 1ns on fd 23.
(D)[06:54:26]{backlight.c:291}  Sensor '/dev/bus/usb/005/002' is now available.
(D)[06:54:26]{backlight.c:497}  Resumed as a sensor is now available.
(D)[06:54:26]{interface.c:455}  Emitting 'DayTime' property
(D)[06:54:26]{interface.c:455}  Emitting 'NextEvent' property
(D)[06:54:26]{interface.c:455}  Emitting 'SensorAvail' property
(D)[06:54:26]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:54:26]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:54:36]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:54:36]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:54:46]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:54:46]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:54:56]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:54:56]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:55:06]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:55:06]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:55:17]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:55:17]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:55:27]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:55:27]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:55:37]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:55:37]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:55:47]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:55:47]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:55:57]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:55:57]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:56:07]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:56:07]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:56:17]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:56:17]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:56:27]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:56:27]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:56:37]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:56:37]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:56:48]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:56:48]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:56:58]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:56:58]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:57:08]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:57:08]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:57:18]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:57:18]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:57:28]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:57:28]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:57:38]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:57:38]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:57:48]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:57:48]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:57:58]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:57:58]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:58:08]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:58:08]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:58:19]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:58:19]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:58:29]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:58:29]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:58:39]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:58:39]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:58:49]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:58:49]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:58:59]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:58:59]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:59:09]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:59:09]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:59:19]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:59:19]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:59:29]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:59:29]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:59:39]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:59:39]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[06:59:50]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[06:59:50]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[07:00:00]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[07:00:00]{timer.c:32}   Set timeout of 10s 0ns on fd 23.
(D)[07:00:10]{bus.c:181}    capture_frames_brightness(): Input/output error
(D)[07:00:10]{timer.c:32}   Set timeout of 10s 0ns on fd 23.

`

FedeDP commented 3 years ago

(D)[06:54:26]{backlight.c:291} Sensor '/dev/bus/usb/005/002' is now available.

It seems like Clight doesn't see your /dev/video0 webcam; instead it tries to use Yocto sensor (i guess). Can you check busctl call org.clightd.clightd /org/clightd/clightd/Sensor org.clightd.clightd.Sensor Capture "sis" "" 5 "" output?

hunterzero99 commented 3 years ago

I'm actually trying to use the Yocto sensor and not a webcam (I don't even have a webcam, I have Oculus Rift sensors but i just unplugged them and it still isn't working).

Output:

busctl call org.clightd.clightd /org/clightd/clightd/Sensor org.clightd.clightd.Sensor Capture "sis" "" 5 ""
sad "/dev/bus/usb/001/007" 5 0.044 0.044 0.044 0.044 0.044
FedeDP commented 3 years ago

I don't even have a webcam, I have Oculus Rift sensors but i just unplugged them and it still isn't working)

Wow so above output:

busctl call org.clightd.clightd /org/clightd/clightd/Sensor/Camera org.clightd.clightd.Sensor Capture "sis" "" 5 "" sad "/dev/video0" 5 0.156863 0.156863 0.156863 0.156863 0.156863

was from Oculus Rift! Nice :D

Fact is, Clightd gives higher priority to ALS devices and Yocto devices, thus, even with Oculus connected, Clight tries to use Yocto instead of webcam. The problem here is that Yocto is not working fine.

busctl call org.clightd.clightd /org/clightd/clightd/Sensor org.clightd.clightd.Sensor Capture "sis" "" 5 "" sad "/dev/bus/usb/001/007" 5 0.044 0.044 0.044 0.044 0.044

This means it is returning 0 as ambient brightness.

Yesterday instead, you posted:

busctl call org.clightd.clightd /org/clightd/clightd/Sensor org.clightd.clightd.Sensor Capture "sis" "" 5 "" sad "/dev/bus/usb/005/005" 5 0.0976 0.0976 0.0976 0.0976 0.0976

thus it seemed working. Can you double-check that busctl call org.clightd.clightd /org/clightd/clightd/Sensor org.clightd.clightd.Sensor Capture "sis" "" 5 "" always returns "0.044 0.044 0.044 0.044 0.044" ?

Finally,

(D)[06:58:49]{bus.c:181} capture_frames_brightness(): Input/output error

this means that sensor is timing out without doing any capture of your ambient brightness (ie: you requested 5 captures, and it returned 0).

Btw what YoctoLight device you have? Ie: Yoctolight v3? I own a v3 device.

Maybe your kernel was recently upgraded and you still have to reboot? That could explain yocto sensor issues, i guess.

hunterzero99 commented 3 years ago

🤦 I rebooted and now it is working perfectly!

I can't thank you enough for all of your help, and the fixes!

FedeDP commented 3 years ago

Yuppi!! "Turn it off and on again", best tech advice since... always :D Almost guaranteed to work!