Extend container support to all Tier 1 platforms #43
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
EdwardSnyder-NOAA
force-pushed
the
develop
branch
from
eff0ec6
to
1b77d1f
Compare
|
The working branch diverged too much from the head. Will re-open/create new PR when ready. |
|
PR-85 was created in place of this PR. |
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.
Description
This PR extends support for the Land DA develop container to all Tier 1 platforms. To accomplish this, a new script was added to the ufs-containers repo and documentation changes are needed.
Most T1 platforms have Intel MPI installed and can run the workflow as described in the documentation with the exception of Derecho, Gaea, and Hercules. For these platforms different Intel compilers and MPIs are used, which is described in this PRs document changes. In addition, the
setup_container.shfrom the ufs-containers repo needs to run on Derecho and Gaea. For Derecho, this script changes the slurm commands to the pbs equivalent commands because that is the job scheduler used there. And for Gaea, this script switches the mpiexec commands to srun and adds additional sbatch commands so that the jobs can run there.The container was tested with both the era5 and gswp3 cases for all T1 platform. I can provide the locations of these tests if needed.
Anticipated changes to regression tests:
Subcomponents involved:
Linked PR's and Issues:
Testing (for CM's):