-
Notifications
You must be signed in to change notification settings - Fork 209
Anvil and Delta supercomputer GPU node config examples in documentation #3904
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
Conversation
…on both Anvil and Delta supercomputers.
Whoops, let me fix these failed tests... |
Alright now I'm not so sure what might be causing these test failures... Any input is welcome. |
race conditions either in or exposed in parsl by, the test suite - i hit rerun. |
Great, thank you. |
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.
lgtm (though I didn't test the configs)
@lvotapka - this is probably a silly question, but both the configs you are adding have been tested? |
@danielskatz , I have run my own versions of these, within Python scripts that included some string formatting, with my own account ID, etc., although they were not in exactly this same format. Do you want me to try and run these exact scripts on these hosts? |
If you could, that would be great |
Alright I tested both of these and they work correctly, @danielskatz . |
@benclifford | @yadudoc - can you merge this? |
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.
Wonderful, the config options look correct to me.
Thanks for pushing these configs @lvotapka!
@yadudoc, you're welcome! |
Added two new sections to documentation config examples for GPU jobs on both Anvil and Delta supercomputers.
Type of change