Skip to content
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

Add support to monitor CPU temperature #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ffeldbauer
Copy link

Added support to monitor the CPU temperature on Linux. New function uses the LOAD_TYPE scan rate like all the other CPU related informations.

Added support to monitor the CPU temperature on Linux.
New function uses the LOAD_TYPE scan rate like all the
other CPU related informations.
@ffeldbauer
Copy link
Author

ffeldbauer commented Sep 25, 2023

Sorry for not responding so long regarding my merge request #45
I was busy preparing our Detector setup for a beamtime and was not able to look into the issue regarding the broken CI.

When I tried to merge the latest master into my fork, I ran into a few new issues, which is why I started from a fresh clone of the current master and added my changes to it

@JJL772
Copy link

JJL772 commented Dec 2, 2023

/sys/class/thermal/thermal_zone0 on my system corresponds to ACPI interface temperature. you might need to iterate each thermal_zone and check the type file in each of them.
for me, thermal_zone4 had type x86_pkg_temp, which seems to match up with the CPU package temperature displayed by lm-sensors.

@jeonghanlee
Copy link
Contributor

@ffeldbauer Thank you for bring this subject again. And please check the previous discussion at #7 with @ralphlange

You may find his insights on this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants