Fix instances where charging is not recognized (#281) (#285)

* check if charging in laptop with battery

* using both battery and ac adapter to check if charging
also battery status unknown is now recognized as charging
This commit is contained in:
AmitGolden 2021-11-19 09:08:32 +02:00 committed by GitHub
parent 59996baeba
commit 897664aa3d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions

View File

@ -168,10 +168,13 @@ def charging():
computer_type = getoutput("dmidecode --string chassis-type")
if computer_type in ["Notebook", "Laptop", "Convertible", "Portable"]:
# AC adapter states: 0, 1, unknown
ac_info = getoutput(f"grep . {power_dir}A*/online").splitlines()
# Battery statuses: Full, Charging, Discharging, Unknown
battery_status = getoutput(f"grep . {power_dir}B*/status").splitlines()
# if there's one battery charging, ac_state is True
ac_state = any(["Full" in ac or "Charging" in ac for ac in battery_status])
# if there's one battery charging, or if there's one ac-adapter on-line, ac_state is True
ac_state = (any([not "Discharging" in ac for ac in battery_status]) or
any(["1" in ac.split(":")[-1] for ac in ac_info]))
else:
has_battery = psutil.sensors_battery() is not None
if has_battery: