← Back to team overview

screenlets-dev team mailing list archive

[Bug 694857] Re: [Sensors] sensors.py flaws

 

It's good that the memory bug has been corrected all right.

About the CPU usage. Sorry, we cannot just divide the amout with the
number of processors, because I don' think it is meant work this way. I
think that every processor has it's own line in the /proc/stat file
(even if I have one processor, I have two lines) and sensors.py
definetly does not add the lines. The problem should be somewhere else.
If you can provide a link to a standard solution or explanation of the
issue, we can find out the right way to fix it. If you can provide also
a patch, that would be even better. But maybe the first thing could be
that you provide us with "grep ^cpu /proc/stat;sleep 1;grep ^cpu
/proc/stat" or something like that. Do you have a normal unmodified
kernel?

About the temperature indicator. There are lot of temperature indicators
in sensors.py that provide the result with the 'C' added. I don't know
why the GPU temperature is provided as number, there may be some
historical or whatever reasons. But since it has been this way for long
time, I cannot just change the "useless string" format and render other
Screenlets unusable. As a prototype, I tried to implement "hddtemp
sensor-numeric <device>" in additon to "hddtemp sensor <device>". I
wasn't able to test it, so please test if it works (rev 571) and give
your feedback.

-- 
You received this bug notification because you are a member of
Screenlets Dev Team, which is subscribed to Screenlets.
https://bugs.launchpad.net/bugs/694857

Title:
  [Sensors] sensors.py flaws



References