Added CPU temperature information #44
Conversation
whitfijs-jw
commented
Dec 18, 2019
whitfijs-jw
commented
- added retrieval of CPU temperature 'x86_pkg_temp' from /sys/class/thermal/ directory -- this 'should' be the highest coretemp read from hwmon - copy pasta for the indicator and info in the drop down view - added CPU temp to schema - so far only tested on a Sandy Bride laptop -- can probably test on a Haswell and AMD FX in the near future
- fixed issue with string compare on "x86_pkg_temp"
- added try catch for enumerator to satify warnings
|
Should be cleaned up now. I liked the layout of the indicator so I figured I'd add CPU temp. x86_pkg_temp likely isn't going to work for all platforms -- it works on the 2 Intel CPUs computers that I have laying around. The pattern matching for the type in update_temperature in CPU.vala can be extended for other platforms. Thanks again for your work -- hope this helps someone. |
|
Thanks for your work. I will probably pull the branch and test is myself to see how it looks. Will come back with an update. |
|
Hello! Sorry to interrupt, I've tried to use it on my Dell laptop, I'm on latest version of Elementary, but no temperature is showing, I've tried watching in settings but did not found anything.. Am I missing some steps? I've followed the same install procedure in the Readme. Maybe I should remove the previous installation?
|
|
Sorry for the delay, the extension I added depends on finding the type "x86_pkg_temp" in "/sys/class/thermal/thermal_zone*" directory You can list the types like this: If "x86_pkg_temp" isn't a type there let me know and I can add more patterns to support different platforms. |
|
On a fresh install it works. Maybe the problem it's reinstalling on a machine with the plugin already installed |
