-
Notifications
You must be signed in to change notification settings - Fork 13
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
port RDASApp to Gaea #196
Merged
Merged
port RDASApp to Gaea #196
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
update module files, docs and scripts
guoqing-noaa
requested review from
hu5970,
ShunLiu-NOAA,
chunhuazhou,
delippi,
TingLei-NOAA,
SamuelDegelia-NOAA,
spanNOAA and
Junjun-NOAA
October 14, 2024 21:38
rrfsbot
added
running_hercules
Passed_hercules
and removed
test_hercules
running_hercules
labels
Oct 15, 2024
PASSED on hercules started build_and_test on hercules at UTC time: Tue Oct 15 01:45:56 UTC 2024
workdir: /work/noaa/wrfruc/rrfsbot/PRs_RDASApp/196 |
SamuelDegelia-NOAA
previously approved these changes
Oct 15, 2024
@SamuelDegelia-NOAA Thanks for your approval. Could you do it again? |
SamuelDegelia-NOAA
approved these changes
Oct 15, 2024
TingLei-NOAA
approved these changes
Oct 15, 2024
ShunLiu-NOAA
approved these changes
Oct 15, 2024
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.
update module files, docs and scripts
The mpasjedi 2024052700 case can ran through without a problem.
Since GAEA require to specify
-M
directive for any jobs, it will take some time to figure out how to run ctests on Gaea. Nevertheless, this does not affect the review and merge of this PR and we don't plan to run CI tests or ctests on Gaea in the near future (CI tests on three platforms Hera/Jet/Hercules should be enough at this moment).Link to issue #170