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
Port cpu_monitor to ROS2 #326
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for porting this. Please see the inline comments
diagnostic_common_diagnostics/diagnostic_common_diagnostics/cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/diagnostic_common_diagnostics/cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/test/systemtest/test_cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/test/systemtest/test_cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/test/systemtest/test_cpu_monitor.py
Outdated
Show resolved
Hide resolved
…and move the sleep to there to make sure this is done every test in a systemactic way
Thanks for reviewing! I updated the files accordingly :) |
diagnostic_common_diagnostics/diagnostic_common_diagnostics/cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/diagnostic_common_diagnostics/cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/diagnostic_common_diagnostics/cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/diagnostic_common_diagnostics/cpu_monitor.py
Outdated
Show resolved
Hide resolved
diagnostic_common_diagnostics/diagnostic_common_diagnostics/cpu_monitor.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix the failing tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix
- SyntaxError in diagnostic_common_diagnostics/cpu_monitor.py:95
@ct2034 Excuse moi! Everything should be fixed now :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I wanted to make these changes myself but could not push to your branch
@ct2034 did you find time already to re-review this PR? :) |
I attempted to port the cpu_monitor to ROS2. The module allows users to monitor the CPU usage of their system in real time. It publishes the usage percentage in a diagnostic message.
Let me know what you think! :)