-
Notifications
You must be signed in to change notification settings - Fork 816
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
fix pynvml import failure #1882
Conversation
@lxning curious why this was not an issue always and only showed up recently? |
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.
Looks good. Shouldn't the CPU benchmark CI always be failing because of this?
Codecov Report
@@ Coverage Diff @@
## master #1882 +/- ##
==========================================
- Coverage 44.99% 44.95% -0.04%
==========================================
Files 63 63
Lines 2609 2609
Branches 60 60
==========================================
- Hits 1174 1173 -1
- Misses 1435 1436 +1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@rohithkrn This issue happened recently due to PR. It causes CPU usage report different from previous. |
@ankithagunapal I found PR caused cpu usage abnormal, but not sure if it is also related to recent CPU benchmark unstable. |
Description
Please read our CONTRIBUTING.md prior to creating your first pull request.
Please include a summary of the feature or issue being fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #(issue)
#1881
Type of change
Please delete options that are not relevant.
Feature/Issue validation/testing
Please describe the Unit or Integration tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.
Only "import pynvml" in gpu host.
Test A
Logs for Test A
Test B
Logs for Test B
Checklist: