-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CPU Temp suggestion #24
Comments
I'll add this to the list. We've played with this several times before, with every attempt having some kind of shortfall. |
I have fixed this in the last commit, check out. ( at least worked for me) |
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. |
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. |
See what I mean, always some small technicality :p |
Currently I'm trying out
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 |
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.
The text was updated successfully, but these errors were encountered: