friso/rpi-mqtt-monitor
Public Access
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Correctly parse system load on locales with "," as decimal point

Browse Source 
On my RPi with German locale the output of `uptime` looks like this:

` 12:56:38 up 4 days, 15:35,  3 users,  load average: 0,24, 0,24, 0,16`

Default implementation cuts off the decimal point during parsing, resulting in very low CPU load accuracy being reported.

Fix is a single character. Corrected version works both on German and English locales.
This commit is contained in:
Olex S
2021-04-20 12:58:28 +02:00
committed by GitHub
parent b9ebfb1375
commit 73beb0231f
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/rpi-cpu2mqtt.py
View File

@@ -25,7 +25,7 @@ def check_cpu_load():
# bash command to get cpu load from uptime command # bash command to get cpu load from uptime command
p = subprocess.Popen("uptime", shell=True, stdout=subprocess.PIPE).communicate()[0] p = subprocess.Popen("uptime", shell=True, stdout=subprocess.PIPE).communicate()[0]
cores = subprocess.Popen("nproc", shell=True, stdout=subprocess.PIPE).communicate()[0] cores = subprocess.Popen("nproc", shell=True, stdout=subprocess.PIPE).communicate()[0]
cpu_load = str(p).split("average:")[1].split(",")[0].replace(' ', '').replace(',', '.') cpu_load = str(p).split("average:")[1].split(", ")[0].replace(' ', '').replace(',', '.')
cpu_load = float(cpu_load)/int(cores)*100 cpu_load = float(cpu_load)/int(cores)*100
cpu_load = round(float(cpu_load), 1) cpu_load = round(float(cpu_load), 1)
return cpu_load return cpu_load