battery-stats icon indicating copy to clipboard operation
battery-stats copied to clipboard

Published 20 hours ago • verified publisher icon petterreinholdtsen

Support multiple batteries

Open mark7 opened this issue 6 years ago • 2 comments

Previously, only data on the first battery found was collected. Some laptops (like the Thinkpad T470) have multiple batteries, resulting in the charge from only one being logged.

Treat all batteries as one large "aggregate" battery and store data on its charge.

mark7 avatar Apr 28 '18 16:04 mark7

I would imagine it make more sense to record information about each battery individually. Why do you believe it make sense to store the combined values?

petterreinholdtsen avatar Apr 28 '18 16:04 petterreinholdtsen

Well:

  • Existing software that I use, like i3status, aggregates the charge.

  • Multiple separate batteries would require UI and logging changes (e.g. all the commands would need to be parameterized on individual battery), whereas an aggregate battery permits continuing to use the software as-is. I can't say that there's no chance that someone would have a use for treating them separately (e.g. if someone physically swaps batteries, which is one thing that someone can do with a laptop with multiple batteries), but I simply am interested in how much life the laptop as a whole has remaining -- the location of the individual charge isn't very interesting, but how much longer the laptop as a whole has to run is.

mark7 avatar Apr 29 '18 05:04 mark7