Porting to pawsey #326
Draft
Porting to pawsey #326
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
linked. Also for the same reason don't assume LD_LIBRARY_PATH is set. Commented out call to load_modules. Pawsey has a lot of default modules that it relies on, so can't reliably monkey with that. Removed a couple of the bespoke flags Marshall added to the slurm scheduler, and also explicitly pass through the PAYU environment variables. Also set in the current environment, but that didn't seem to make it through to the submitted job.
|
Hello @aidanheerdegen! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2022-05-26 04:57:08 UTC |
|
This is a place-holder to make it easier to find and focus attention on what is required to port to magnus @ pawsey. Some of the changes are only to get it working, and aren't intended to be merged into |
This was referenced Apr 8, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Wrap ldd in try/except as executables on pawsey seem to be statically linked.
Also for the same reason don't assume LD_LIBRARY_PATH is set.
Commented out call to load_modules. Pawsey has a lot of default modules
that it relies on, so can't reliably monkey with that.
Removed a couple of the bespoke flags Marshall added to the slurm
scheduler, and also explicitly pass through the PAYU environment
variables. Also set in the current environment, but that didn't seem
to make it through to the submitted job.
Closes #323