ACPI shows average battery power when mouse with battery added about battery.nvim

[justinhj]

@FMMazur thank you for submitting this issue.
This relates to #7
This is interesting; I assumed that if a user had multiple batteries it would be okay to show the average, assuming they are all laptop power. But I see that other batteries show up and it makes no sense. I will make an update soon to show only the first battery power.

[justinhj]

Yes just updated it this morning. Please grab the latest or set the tag to v.0.5.0 and let me know if it is fixed. The new behaviour takes the first battery found.

[FMMazur]

I found something in MSDN, it says, the device can send the _SUN method as specified on ACPI Reference. So it can be a firmware that don't send that optional method or it could be some driver problem as related at this reddit post.

[netfri25]

Now it works perfectly!! Thanks a lot, I truly appreciate the effort!

[netfri25]

Is there an update on this issue? I have the exact same problem

[FMMazur]

I believe that the way it is implemented in the commit may not always works, I will check it later.

[justinhj]

I may have to make it either display multiple batteries or pick one based on some criteria. Let me know if you see any documentation on how acpi reports.
Another option, which is less satisfying for the user, would be to have a configurable battery number (take the 1st, 2nd or 3rd and so on)

[netfri25]

It still has some issues.
The order of the batteries is not always the same, which makes it sometimes show 0%

[netfri25]

I don't really know how to program in Lua so I have no idea how to implement this, but in /sys/class/power_supply/BAT0/ there is a file called capacity which stores the battery percentage

[justinhj]

@netfri25 thanks for the info. I think the problem with using the file system is that it varies across different linux distributions, whereas the if I require the user to install acpi that levels the playing field. Unfortunately I don't have any idea how acpi orders the batteries so not sure how to pick one.

[justinhj]

I'm going to add a new configuration variable 'multiple_battery_selection'

1. "max" will choose the battery with the highest percentage power
2. "avg" will average the battery percentages
3. 1,2,3... will choose the first, second, third battery reported
   It will default to 1 so the existing behaviour will be unchanged.
   This will initially only be implemented for linux/acpi.

[justinhj]

I've implemented the multiple_battery_selection option in the branch battery-chooser.

battery.setup({
	update_rate_seconds = 30,           -- Number of seconds between checking battery status
	show_status_when_no_battery = true, -- Don't show any icon or text when no battery found (desktop for example)
	show_plugged_icon = true,           -- If true show a cable icon alongside the battery icon when plugged in
	show_unplugged_icon = true,         -- When true show a diconnected cable icon when not plugged in
	show_percent = true,                -- Whether or not to show the percent charge remaining in digits
    vertical_icons = true,              -- When true icons are vertical, otherwise shows horizontal battery icon
    multiple_battery_selection = 1,     -- Which battery to choose when multiple found. "max" or "maximum", "min" or "minimum" or a number to pick the nth battery found (currently linux acpi only)
})

I don't have a linux machine handy to test this on so I'm not ready to push to main, but if anyone can verify that it works and helps with this issue I will do so.

[justinhj]

@netfri25 in your case I would set multiple_battery_selection = max, so that you always get the battery that is charged not the 0%.