Enable inline post.#27
Merged
christinaholtNOAA merged 12 commits intoApr 15, 2021
Merged
Conversation
Change platform from aws.intel to aws. Change Intel MPI module to use the module that supports EFA.
christinaholtNOAA
requested review from
christopherwharrop-noaa and
danielabdi-noaa
christopherwharrop-noaa
approved these changes
Apr 15, 2021
Collaborator
christopherwharrop-noaa
left a comment
christopherwharrop-noaa
left a comment
There was a problem hiding this comment.
Only nitpick is the removal of newline in the env/build_aws_intel.env file.
Collaborator
|
Should we maybe add loading of the correct rocoto in |
Collaborator
Author
|
I can absolutely add that. We haven't historically done that in the repo, but this branch is all about making our lives easier for this project. :) |
Collaborator
Author
|
Just to double check, we want this in there, right? |
Collaborator
Yes. |
danielabdi-noaa
approved these changes
Apr 15, 2021
christopherwharrop-noaa
approved these changes
Apr 15, 2021
Ruifang-Li
pushed a commit
to Ruifang-Li/ufs-srweather-app
that referenced
this pull request
Sep 23, 2021
- Adds a CMake build using ExternalProject_Add.
- Makes modifications to build_all.sh and install_all.sh for back-compatibility with old build system. The old build system will remain in, but is now deprecated, and will be removed in the coming weeks.
To build, have the necessary modules loaded and run:
```
mkdir build
cd build
cmake .. -DCMAKE_INSTALL_PREFIX=<prefix>
make
```
The docs/ directory contains README files with instructions for specific platforms/compilers
- **Cheyenne**: Build and end-to-end test ("DOT_OR_USCORE" test case) was successful on Cheyenne with intel, both for the cmake build and the old build script (that will soon be deprecated). Path to tests: /glade/scratch/kavulich/UFS_CAM/testing/SRW_PR_27/expt_dirs/
- **Hera**: Build and end-to-end tests successful (aside from expected failures). Path to tests: /scratch2/BMC/det/kavulich/workdir/SRW_PR_27/expt_dirs
- **Jet**: Build test was successful.
Resolves NOAA-GSL#7, partially resolves NOAA-GSL#5
Co-authored-by: kgerheiser <kgerheiser@icloud.com>; Michael Kavulich, Jr <kavulich@ucar.edu>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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 OF CHANGES:
This change allows for the use of the Inline Post with the forecast model, and removes the EMC_Post externals/build. It sets the subset of physics packages that we want to run for the HWT. Lastly, it points to a UFS_UTILS branch that has a fix for running successfully with GEFS initial and boundary conditions.
Dependencies
This set of mods should be merged along with a PR in the regional_workflow repository.
TESTS CONDUCTED:
These changes were tested by @christopherwharrop-noaa and @danielabdi-noaa on AWS, and by Christina on Hera.
CONTRIBUTORS (optional):
@christopherwharrop-noaa @danielabdi-noaa