search

epics-modules / iocStats

EPICS IOC Status and Control
Other
12 stars 40 forks source link

Add support to monitor CPU temperature #45

Closed ffeldbauer closed 1 year ago

ffeldbauer commented 2 years ago

Somehow I was missing the CPU temperature in this module, so I tried to add it at least for Linux. I implemented the function like the CPU utilization and used the same scan rate (LOAD_TYPE) for it

simon-ess commented 1 year ago

LGTM, but note that there are now some conflicts with README_devIocStats. If you can rebase this to master that should be able to address that.

ffeldbauer commented 1 year ago

Yes, I have noticed. Fixed with newest commit

simon-ess commented 1 year ago

I did add a CI pipeline to this, and it seems that your resolution to the issue has undone that.

ffeldbauer commented 1 year ago

Not sure, why the submodule was not merged correctly. Due to some new merge conflicts with the latest master, I created a new "fork". Since I cannot edit the source branch within this pull request, I had to create a new one..... => #56