[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-10 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=347470

Kai Uwe Broulik  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
  Latest Commit||http://commits.kde.org/powe
   ||rdevil/d3d2ebcfd214a892ac79
   ||484cfb8cb6fcbf7fc917
   Version Fixed In||5.3.1
 Resolution|--- |FIXED

--- Comment #8 from Kai Uwe Broulik  ---
Git commit d3d2ebcfd214a892ac79484cfb8cb6fcbf7fc917 by Kai Uwe Broulik.
Committed on 10/05/2015 at 16:54.
Pushed by broulik into branch 'Plasma/5.3'.

Consider overall percentage when emitting battery warning on startup

Otherwise you might have your session shutdown after logging in when the first
battery
that was detected happened to be below the critical threshold.

REVIEW: 123704
FIXED-IN: 5.3.1

M  +4-9daemon/powerdevilcore.cpp

http://commits.kde.org/powerdevil/d3d2ebcfd214a892ac79484cfb8cb6fcbf7fc917

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=347470

Kai Uwe Broulik  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |CONFIRMED

--- Comment #7 from Kai Uwe Broulik  ---
Thank you!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Philipp Rehs
https://bugs.kde.org/show_bug.cgi?id=347470

--- Comment #6 from Philipp Rehs  ---
I just compiled powerdevil-git with this patch and it seems to fix the problem
in a short test.
I will run it the next days and give you some feedback

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=347470

--- Comment #5 from Kai Uwe Broulik  ---
If you can, please try this patch: https://git.reviewboard.kde.org/r/123704/

I'm afraid, no, the order is determined by the system (whatever battery comes
first), at least for mouse batteries it shows the device name but internal
batteries usually have cryptic serial numbers and so it just shows them
numbered.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Philipp Rehs
https://bugs.kde.org/show_bug.cgi?id=347470

--- Comment #4 from Philipp Rehs  ---
Your idea could be right. This time it happened when the second battery was
empty.

off-topic:
is there any way to give a priority to batteries? or give them differend names?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Philipp Rehs
https://bugs.kde.org/show_bug.cgi?id=347470

--- Comment #3 from Philipp Rehs  ---
philipp@philipp-t440s ~ $ upower -d
Device: /org/freedesktop/UPower/devices/line_power_AC
  native-path:  AC
  power supply: yes
  updated:  Sa 09 Mai 2015 16:46:53 CEST (3 seconds ago)
  has history:  no
  has statistics:   no
  line-power
warning-level:   none
online:  no
icon-name:  'ac-adapter-symbolic'

Device: /org/freedesktop/UPower/devices/battery_BAT0
  native-path:  BAT0
  vendor:   SONY
  model:45N
  serial:   10860
  power supply: yes
  updated:  Sa 09 Mai 2015 16:46:53 CEST (3 seconds ago)
  has history:  yes
  has statistics:   yes
  battery
present: yes
rechargeable:yes
state:   discharging
warning-level:   none
energy:  15,34 Wh
energy-empty:0 Wh
energy-full: 19,29 Wh
energy-full-design:  23,2 Wh
energy-rate: 13,075 W
voltage: 11,957 V
time to empty:   1,2 hours
percentage:  79%
capacity:83,1466%
technology:  lithium-polymer
icon-name:  'battery-full-symbolic'

Device: /org/freedesktop/UPower/devices/battery_BAT1
  native-path:  BAT1
  vendor:   LGC
  model:45N1127
  serial:   5990
  power supply: yes
  updated:  Sa 09 Mai 2015 16:46:53 CEST (3 seconds ago)
  has history:  yes
  has statistics:   yes
  battery
present: yes
rechargeable:yes
state:   discharging
warning-level:   low
energy:  4,44 Wh
energy-empty:0 Wh
energy-full: 21,13 Wh
energy-full-design:  23,48 Wh
energy-rate: 26,616 W
voltage: 11,295 V
time to empty:   10,0 minutes
percentage:  21%
capacity:89,9915%
technology:  lithium-ion
icon-name:  'battery-low-symbolic'
  History (charge):
1431182813  21,000  discharging
1431182801  20,000  charging
  History (rate):
1431182813  26,616  discharging
1431182813  26,862  discharging
1431182804  26,616  charging
1431182801  26,574  charging

Device: /org/freedesktop/UPower/devices/DisplayDevice
  power supply: yes
  updated:  Sa 09 Mai 2015 16:46:53 CEST (3 seconds ago)
  has history:  no
  has statistics:   no
  battery
present: yes
state:   discharging
warning-level:   none
energy:  19,78 Wh
energy-full: 40,42 Wh
energy-rate: 39,691 W
time to empty:   29,9 minutes
percentage:  48,9362%
icon-name:  'battery-good-symbolic'

Daemon:
  daemon-version:  0.99.2
  on-battery:  yes
  lid-is-closed:   no
  lid-is-present:  yes
  critical-action: HybridSleep

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=347470

Kai Uwe Broulik  changed:

   What|Removed |Added

 Status|NEEDSINFO   |UNCONFIRMED
 Resolution|WAITINGFORINFO  |---

--- Comment #2 from Kai Uwe Broulik  ---
Ah, no, I misunderstood your bug report. So apparently when the system wakes up
or starts, then one battery is added (well, it iterates over all of them
initially), examined, found to be low of charge, and the next one that appears,
doesn't cancel the shutdown process. Makes sense, I'll have a look.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=347470

Kai Uwe Broulik  changed:

   What|Removed |Added

 Resolution|--- |WAITINGFORINFO
 CC||k...@privat.broulik.de
 Status|UNCONFIRMED |NEEDSINFO

--- Comment #1 from Kai Uwe Broulik  ---
Are both batteries recognized by the system? Can you provide the output of
upower -d? 
Quickly skimming through the code, when a battery changes its percentage, it
calculates the overall percentage and when that one is below the configured
threshold, it emits a warning and starts the shutdown timer. I don't see why it
would not work in your case :/ I don't have the hardware to test this,
unfortunately.
Also, it might make sense to add a "Cancel" button to the notification, if sbd
is willing to take the risk of data loss.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[Powerdevil] [Bug 347470] Shutdown when one of two batteries is empty

2015-05-09 Thread Philipp Rehs
https://bugs.kde.org/show_bug.cgi?id=347470

Philipp Rehs  changed:

   What|Removed |Added

   Platform|Other   |Archlinux Packages
 OS|other   |Linux
 CC||p.r...@gmx.net

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel