-
Notifications
You must be signed in to change notification settings - Fork 18
WIP: Add computer_hw package (copied, improved from pr2_computer_monitor) #20
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
Closed
Conversation
This file contains hidden or 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
…ericity. Move all .launch files to conf folder.
…d so that diagnostics analyzer and other downstream utils ignore
…tially any container).
Add computer_hw package (copied and improved from pr2_computer_monitor)
Member
Author
Not yet sure why one of the CI jobs failed.https://app.travis-ci.com/github/ros-drivers/linux_peripheral_interfaces/jobs/538168773#L296 |
130s
added a commit
to kinu-garage/linux_peripheral_interfaces
that referenced
this pull request
Sep 19, 2021
Member
Author
|
Not familiar with these Py error yet but https://stackoverflow.com/questions/57475673 suggests the code is already Py3, and this CI job wants see the code is compatible with Py2? |
Member
Author
I can see locally the same though. |
Closed
3 tasks
130s
referenced
this pull request
in PR2/pr2_robot
Feb 24, 2022
fixup for 7d74c2e there is one use of `print(str, file=...)` after the transition.
2 tasks
Member
Author
|
Closing in favor of #21 |
130s
added a commit
to kinu-garage/linux_peripheral_interfaces
that referenced
this pull request
Feb 16, 2023
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.
Issue aimed at
Changes
computer_hwpackage (renamedpr2_computer_monitorthat was copied from pr2_robot repo).launchto allow downstream to start processes by batch.Review items
Test
Dev test done on Ubuntu 16.04 host with nvidia GeForce GTX 1060
Sample of Diagnostic GUI with GPU monitoring output.
