jjorge marked an inline comment as done.
jjorge added a comment.

  > Are states > 100 or < 0 really well defined? Is it save to assume that a 
state > 100 can be associated with 100?
  
  Yes : this is a percentage. States out of this value are bad hardware : I 
have a travel battery which shows crazy values along time, from 0,001Wh 
capacity till 901Wh (Real capacity is near 45Wh).
  
  > This should maybe be a part of a separate patch as it seems unrelated.
  
  You're right, this is already in master, I have badly also selected this 
commit in my patch.

REPOSITORY
  R106 KSysguard

BRANCH
  acpi-move-battery-to-sysfs

REVISION DETAIL
  https://phabricator.kde.org/D25018

To: jjorge, davidedmundson, #plasma, ahiemstra
Cc: alexde, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, 
GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to