feat: add red/blue temperature color coding#170
Conversation
a5000c8 to
0c05d4e
Compare
hamirmahal
left a comment
There was a problem hiding this comment.
With this change
Without this change
|
bvaisvil
left a comment
There was a problem hiding this comment.
Thanks for the contribution! I'm curious at how we decide what a set of good thresholds are here.
| .join(","); | ||
| format!(" TEMP [{t:}°C]") | ||
|
|
||
| let hot_threshold = 70_f64; |
There was a problem hiding this comment.
Curious how these thresholds were determined. I imagine that there is also some processor variety in acceptable temperatures ?
There was a problem hiding this comment.
That's a fair point.
https://hone.gg/blog/best-cpu-temperature-for-pc/ shows 40 degrees Celsius on the left, safe side of the spectrum.
https://blog.pcbuilder.net/ideal-cpu-temperature/ says
You should always keep it consistent at 60-70 degrees Celsius; otherwise, the heat may damage your CPU.
While in general use, when you use heavy applications, you should have the temperature of your CPU limited within 40-70 degrees Celsius. This will offer you an extended life span of your CPU.
However, based on the average of 30 distinct processors, the typical normal temperature range is 61 degrees to 73 degrees Celsius.
The secure CPU temperature range for Intel and AMD desktop CPUs is 45-50 degrees Celsius while idle
There was a problem hiding this comment.
I think longer term this should be part of a config along with the color choices. But this is fine for now.
|
You're welcome! I left some context at #169 (comment). |
Closes #169