search

victronenergy / gui-v2

Other
17 stars 6 forks source link

Adaptive brightness makes the backlight pulse #1033

Closed mpvader closed 3 weeks ago

mpvader commented 4 weeks ago

Seen by Lars F. (contact of me, Matthijs).

report:

Hey, latest venus beta on cerbo gx: adaptive brightness can’t switch to off. it’s on again when going back an forth to the display menu. display is pulsing every few seconds. i have recorded a video.

That latest beta he has is v3.40~10; thats a somewhat older gui-v2 version.

https://vrm.victronenergy.com/installation/373064/dashboard

https://github.com/victronenergy/gui-v2/assets/5200296/97426179-6f64-4726-988e-e0c6a4edd59a

https://github.com/victronenergy/gui-v2/assets/5200296/3b185514-d072-4bdf-a23d-ca62e266a3d9

jpetrell commented 3 weeks ago

adaptive brightness can’t switch to off. it’s on again when going back an forth to the display menu.

I can reproduce with v3.40~10, but not on the latest gui-v2. Duplicate to #955, where switch toggling didn't work properly when tapping the switch list item directly on top of the switch indicator. Fixed in gui-v2 v0.2.17 / Venus OS v3.40~~11.

display is pulsing every few seconds.

Pulsation is likely a different issue, gui-v2 doesn't actually implement adaptive brightness, just toggles /AutoBrightness and /Brightness configs from Venus platform. Could not reproduce the pulses shown on the customer video (though the brightness steps are a bit choppy when I add/reduce the overall brightness in the room), tested on Cerbo GX 5" or Ekrano GX.

Sounds like the ambient light sensor <-> display brightness logic needs tuning, or the sensor on customer's Cerbo GX is broken/obscured somehow.

jpetrell commented 3 weeks ago

Pulsation is likely a different issue, gui-v2 doesn't actually implement adaptive brightness

Filed a new issue on the Venus side, closing this one.