-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
test_autorunner_gpu_customization assumes visible device gpu 0 #6247
Comments
could you please have a look? |
After a quick search, The use of |
this is becoming a blocking issue... the python implementation should respect the system variable, for example, setting the device to 1 when the visible devices is 0 will cause an error:
|
Fixes #6247 . ### Description It includes two changes: - Project-MONAI/research-contributions#218 - Project-MONAI/research-contributions#213 ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [x] Integration tests passed locally by running `./runtests.sh -f -u --net --coverage`. Signed-off-by: Mingxin Zheng <[email protected]>
still some issues when running the unit tests as part of please see the log https://github.com/Project-MONAI/MONAI/actions/runs/4677925161/jobs/8286015758 |
Looks like errors are in HPO tests. I will take some time to investigate it next week. |
to reproduce this issue with a multi-gpu node:
more logs:
https://github.com/Project-MONAI/MONAI/actions/runs/4530824415/jobs/7980234392
The text was updated successfully, but these errors were encountered: