Resurrection of Indicator Upower

[paulcarroty]

@Eric-H great job.

[paulcarroty]

New release with many fixes.

[paulcarroty]

Yo, power monitoring and fast charging detection are now supported and I need beta testers (again!) to check how well it works.

You need to install the latest build from Github Actions and start charging your device. If something is wrong - submit your power.txt.

[Eric H]

@paulcarroty
I installed latest build, and the app no longer starts.

https://dpaste.com/6CTBE25DB

[paulcarroty]

@Eric-H fixed.

[Eric H]

@paulcarroty
Reinstalled.
Does not work on surya.

https://dpaste.com/G66MBRASC

[Eric H]

About this error :
ValueError: could not convert string to float: '3,65353'

Maybe it is a localization error (french) : there is a comma instead of a point

[paulcarroty]

@Eric-H yeah, upower depends on current locale. Fixed by enforcing

[Eric H]

@paulcarroty
Reinstalled. It seems to work fine.
Do you need specific tests ?

[paulcarroty]

@Eric-H test fast charging if you have the hardware for.

[Eric H]

@paulcarroty
Sorry, I couldn't do the test before today...

So I used the Xiaomi charger supplied with the phone (which fully charges in less than half an hour) with the latest version (0.7) of uPower, but it seems that fast charging is not detected and there's nothing related in the logs.

[paulcarroty]

Eric-H Attach the power.txt file when the battery is being charged and has less than 50% capacity.

[Eric H]

@paulcarroty
I tried to generate the power.txt file but there is an error on stderr :

cat: '/sys/devices/platform/soc/c440000.qcom,spmi/spmi-0/spmi0-00/c440000.qcom,spmi:qcom,pm6150@0:qcom,qpnp-smb5/power_supply/main/health': Invalid argument

The health files exists and is world readable :

phablet@ubuntu-phablet:~/tmp$ ls -al /sys/devices/platform/soc/c440000.qcom\,spmi/spmi-0/spmi0-00/c440000.qcom\,spmi\:qcom\,pm6150@0\:qcom\,qpnp-smb5/power_supply/main/health 
-r--r--r-- 1 system system 4096 mai    6 06:21 /sys/devices/platform/soc/c440000.qcom,spmi/spmi-0/spmi0-00/c440000.qcom,spmi:qcom,pm6150@0:qcom,qpnp-smb5/power_supply/main/health

The error persists even if I try to

• add double quotes,
• add simple quotes
• replace cat with tail

[Eric H]

@paulcarroty
For some unknown reason, today the message on stderr did not appear.

So here is the power.txt file for surya attached.
or see at https://dpaste.org/hCcZO

I can see POWER_SUPPLY_CHARGE_TYPE=Fast in udev battery info, so it seems to be a fast charge.
What should Indicator Upower display ?