-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add nvidia-smi step to GPU runners #913
Comments
We are testing on CPU runners right now so I expect |
@mikemhenry this is more a to-do for later 😅 |
First, is this something I could open a PR on? I am more than willing to help build this out with you all just would want some direction on how I can be of use in getting this running. One thing I have been doing when testing on both GPU and CPU is to just add a step that looks like this: - name: Check for nvidia-smi
run: "nvidia-smi || true" That way you get to see if it works and if not it doesn't completely fail the run. |
Something else I like to add to "fail fast" is |
This is something we are also working on. I have isolated the part of that script for identifying platforms into something that provides different exit codes based on what is being tested (i.e. if CUDA exists in an OpenMM context return), more than happy to make this more broadly available via the |
Ethan rightly recommends we have that to make sure we have a GPU.
The text was updated successfully, but these errors were encountered: