-
Notifications
You must be signed in to change notification settings - Fork 136
Add sanity check script for driver and GPU info #2272
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
base: main
Are you sure you want to change the base?
Conversation
|
This will address some of the gaps identified in #2107 |
Let me know please if you have any input for improvement - this is the initial version of it. |
Are you able to print out the max VRAM on the GPU device? |
b7e0961 to
ce89a7c
Compare
|
@jayhawk-commits, please see my answers:
I can do that as a follow up. I know we have that info from
Do you mean the identical GPUs or different types of GPUs? For different types (if the case exists), I can do it in the follow up. For the identical ones, it is already supported.
The tests are not running yet for Example output from CI: |
I think we need to be able to support both cases. If there are multiple identical GPUs in the system, we need to report how many. |
|
Is there a label or change we can add to this PR @geomin12 to make tests run on Linux gfx1151? I want to see the APU behaviour. |
ran here :) |
This script is not run during the sanity test step from what I see. Only on the test project steps. |
ah shoot i didn't even read the code and answered! my bad, yeah unfortunately the 1151 linux is only running sanity checks (due to only 4 machines available) you could add this check to |
I think calling it from test_sanity_check would be a good idea. Will check it now. |
Motivation
Progress on: #2209
Test Plan
Test on local and CI
Test Result
Local test result:
CI testing in progress...
Submission Checklist