conky_colors icon indicating copy to clipboard operation
conky_colors copied to clipboard

CPU Temp suggestion

Open jdarnold opened this issue 12 years ago • 6 comments

I suggest changing the last sed when getting the cpu temperature to get rid of a .x temps. So it would be:

sensors -f | grep 'Core 0' | awk '{print $3}' | sed 's/+//' | sed 's/..//g'

instead of s/.0//g at the end.

jdarnold avatar Sep 26 '12 15:09 jdarnold

I'll add this to the list. We've played with this several times before, with every attempt having some kind of shortfall.

android-808 avatar Sep 26 '12 22:09 android-808

I have fixed this in the last commit, check out. ( at least worked for me)

helmuthdu avatar Sep 27 '12 18:09 helmuthdu

Testing here is a little harder, lm-sensors/coretemp never worked right. For example it has just gone 28 to 6 in a second, but hey at least it's not negative today.

The only issue I'm seeing at the moment is the lack of oC or oF.

android-808 avatar Sep 28 '12 08:09 android-808

Hmm, I don't think that is right. Looks like the sed got changed to :

sed 's/.0.*//'

which says to match any a string that is any character followed by a zero, followed by all the rest of the characters.

jdarnold avatar Sep 28 '12 15:09 jdarnold

See what I mean, always some small technicality :p

android-808 avatar Sep 28 '12 20:09 android-808

Currently I'm trying out

sed 's/.[0-9]//g'

This should match any '.' and any following numbers only, leaving the temperature units in place. Please test it as my laptop isn't correctly supported by lmsensors. Either that or my cooling system is ridiculously good inside (4oC now, -2 on boot)

Only issue is it breaks alignment

android-808 avatar Dec 17 '12 12:12 android-808