Bug 342597

Summary: PowerDevil dataengine empty
Product: [Plasma] plasmashell Reporter: Hrvoje Senjan <hrvoje.senjan>
Component: DataEnginesAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: grave CC: gaantonio, kde, kde, me
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.2.1

Description Hrvoje Senjan 2015-01-07 19:09:42 UTC
which in returns provides no brightness controls in the battery applet. as i indicated in the review 121355, this happens since that commit.

Reproducible: Always
Comment 1 Kai Uwe Broulik 2015-01-07 20:43:24 UTC
It's just that it doesn't show up in plasmaengineexplorer (not initially, you can still manually ask it to get the data). Brightness controls do work in battery just fine here. Is kded and powerdevil running?
Comment 2 Hrvoje Senjan 2015-01-07 20:53:32 UTC
off course =)
Comment 3 Vishesh Handa 2015-01-08 14:27:35 UTC
Cannot confirm. Works for me.
Comment 4 Hrvoje Senjan 2015-01-08 15:07:14 UTC
i suspect it is working fine with XRandrBrightness, but not with helper:

powerdevil: No outputs have backlight property
powerdevil: Using helper
Comment 5 Kai Uwe Broulik 2015-01-08 22:11:30 UTC
The only thing I could imagine is that the maximum brightness values aren't populated when it uses the helper. However, all the data engine does is query stuff over DBus from PowerDevil. Also, when I manually request the PowerDevil source it shows up in plasmaengineexplorer just fine. It's just that it doesn't monitor that one because it isn't filled initially (since the aforementioned commit it's filled completely async), when I call setData("PowerDevil", "foo") on powermanagementengine startup it shows up initially just fine.

Can you check the output of "qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/BrightnessControl org.kde.Solid.PowerManagement.Actions.BrightnessControl.brightnessValueMax" ?
Comment 6 Hrvoje Senjan 2015-01-09 01:39:25 UTC
the output is 9
Comment 7 Antonis G. 2015-01-27 13:52:15 UTC
I can reproduce. the output here is 10.
Comment 8 David Edmundson 2015-02-03 09:12:03 UTC
This stuff changed recently in powerdevil, is this still a problem?
Comment 9 Hrvoje Senjan 2015-02-03 11:37:14 UTC
yeah, sorry, forgot to close this one.