Resurrection of Indicator Upower

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 ?