Add cpu temperature alarm #1217
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For issue- cpu temperature alarm
cbrxyz
requested changes
Jul 20, 2024
There was a problem hiding this comment.
Great first attempt! The method of getting the actual CPU temperatures is correct, but it may need a little more work to integrate it into ros_alarms. Please try adding an alarm handler if possible.
|
Hello, it's your friendly InvestiGator bot here! The docs preview for this PR is available at https://mil.ufl.edu/pr-docs/1217. Last updated at:
► 91ead7e929b34c85d0a9565a4a8355a371c83e6b
► 2024-07-31 18:19 EDTHave a great day! Go gators! 🐊 |
|
Sorry, I forgot that you had done the same feature and I implemented a solution in #1233... apologies! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For issue- cpu temperature alarm
Description
I created a script for issue of cpu temp alarm
Screenshot or Video
Related Issues
- Closes #1207
Testing
To test it, they can create a catkin make of the files, run roscore, then do rosrun "name_of_file" . They can test it by having the code check what their temperature is, print the output, then change the variable to be above 90 and see if it sets off an alarm. Below is a cmake.txt they can use for the catkin make
CMakeLists.txt
About This PR