Wrong battery status displayed

[skety]

Wrong battery status displayed

[enkore]

Yes, I see and can reproduce this issue. Will fix.

Explanation:
Battery reports CURRENT_NOW=1000 (µA, =1 mA) despite the battery not being discharged or charged.

Fix in my mind: Assert Full-status even with consumption() != 0 given *_FULL = *_NOW or STATUS=Full (which doesn't work with some batteries, that's why we figure out the status by the numbers in the first place, however, with this fix it should work for yet another kind of broken battery controllers...)

[skety]

Ok, many thanks

[skety]

After updating to the latest version, it works fine. Thanks!

[s-hell]

Same Problem here:
$ cat /cats/class/power_supply/BAT0/uevent
POWER_SUPPLY_NAME=BAT0
POWER_SUPPLY_STATUS=Discharging
POWER_SUPPLY_PRESENT=1
POWER_SUPPLY_TECHNOLOGY=Li-ion
POWER_SUPPLY_CYCLE_COUNT=0
POWER_SUPPLY_VOLTAGE_MIN_DESIGN=14800000
POWER_SUPPLY_VOLTAGE_NOW=14413000
POWER_SUPPLY_CHARGE_FULL_DESIGN=5200000
POWER_SUPPLY_CHARGE_FULL=4958000
POWER_SUPPLY_CHARGE_NOW=2653000
POWER_SUPPLY_CAPACITY=53
POWER_SUPPLY_CAPACITY_LEVEL=Normal
POWER_SUPPLY_MODEL_NAME=BAT
POWER_SUPPLY_MANUFACTURER=NOTEBOOK
POWER_SUPPLY_SERIAL_NUMBER=0001

status.register("battery",
battery_ident="BAT0",
format="{status} {percentage_design:.1f}%",
alert=True,
alert_percentage=10,
color="#008800",
charging_color="#008800",
full_color="#00ff00",
status={
"DPL": "X",
"DIS": "↓",
"CHR": "↑",
"FULL": "=",
},)

Running the current version.

[enkore]

What's the output of i3pystatus in this case?

[s-hell]

It says FULL

[enkore]

Ah, I guess it's because there is no information available about current / power drawn currently. But it should at least get the state right... (since BatteryCharge.consumption() = -1 if no CURRENT_NO is available)

[s-hell]

It doesn't matter if i have one percent ore 90 percent. It always says FULL

[enkore]

This should fix it. Before there was some miscommunication between the layers by using "-1" as a "nope, don't have that" value.

[s-hell]

Looks good!