After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 728971 - Backlight levels differ going from top-to-bottom vs bottom-to-top
Backlight levels differ going from top-to-bottom vs bottom-to-top
Status: RESOLVED WONTFIX
Product: gnome-settings-daemon
Classification: Core
Component: power
3.12.x
Other Linux
: Normal minor
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2014-04-25 20:04 UTC by Conley Moorhous
Modified: 2019-02-01 14:15 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Conley Moorhous 2014-04-25 20:04:53 UTC
The backlight levels from top-to-bottom proceed as follows: 100,86,72,57,43,29,15,0.

[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepDown
(86,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepDown
(72,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepDown
(57,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepDown
(43,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepDown
(29,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepDown
(15,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepDown
(0,)


From bottom-to-top they are different: 0,14,28,43,57,71,85,100
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepUp
(14,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepUp
(28,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepUp
(43,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepUp
(57,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepUp
(71,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepUp
(85,)
[conley@styrka ~]$ gdbus call --session --dest org.gnome.SettingsDaemon.Power
--object-path /org/gnome/SettingsDaemon/Power --method
org.gnome.SettingsDaemon.Power.Screen.StepUp
(100,)

This has to do with the rounding apparently, though I haven't found the error yet. This would be a good Gnome Love/Papercuts bug.
Comment 1 Bastien Nocera 2014-05-19 10:09:23 UTC
How is this actually a problem? A 1% difference in backlight is hardly visible.
Comment 2 Conley Moorhous 2014-05-19 15:30:48 UTC
It's simply a matter of polish; it would be a good candidate for a papercuts or gnome-love bug. It would be nice to have consistent behavior going both ways. It seems obvious to point out that reversible behavior *should* behave in a consistent way. You can close the bug if you would like, though.
Comment 3 Benjamin Berg 2019-02-01 14:15:36 UTC
Probably something odd happening with the rounding. The code has changed considerably, but I suspect that this may still happen.

Anyway, I don't feel like this is problematic enough to take action. If anyone wants to work on ensuring we round/step consistently, then feel free to submit a patch. Please note, that I believe it will be a lot less relevant on modern machines as we have a lot of brightness steps there.