-
Notifications
You must be signed in to change notification settings - Fork 150
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 the GSI to Acorn #793
Port the GSI to Acorn #793
Conversation
Install and build
All tests pass! Note: Use |
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.
some requested changes in gsi_acornn.intel.lua
and optional comments on sub_acorn
.
@DavidHuber-NOAA , @aerorahul is on leave until 11/4. Is there anyone else who can complete the second peer review? Today I will install |
@BinLiu-NOAA @ZhanZhang-NOAA Since you both had helped with the original HAFS GSI port to Acorn, would one of you mind reviewing this PR? |
Cactus (WCOSS2) ctest results
Hera ctest results
Hercules ctest results
Orion ctest results
global_4denvar failed due to
hafs_4denvar_glbens failed for the same reason
Neither of these are fatal failures. A rerun of the ctests yields a Passed result for all tests
Summary |
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.
Approve.
@DavidHuber-NOAA , if you have addressed @aerorahul 's review comments, please go ahead and resolve the conversations. Thanks. ctests have been run on all platforms with expected results. Once we get a second peer review and approval we can schedule this PR for merger into |
@BinLiu-NOAA @ZhanZhang-NOAA : If you have time, your review of this PR is much appreciated. Closure of this PR is a prerequisite to making progress on several open GSI issues. Hence my reaching out to you. |
We would like to merge this PR into |
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.
Looks good to me. And thanks for working on this and enabling Acorn support!
Thank you @BinLiu-NOAA for your approval. @DavidHuber-NOAA , I'll reach out to the GSI Handling Review team to schedule merger of this PR into |
Description
This ports the GSI to Acorn to allow testing of new library installations before official installations go to Cactus and Dogwood. This is a prerequisite (but does not fix) for #642, #662, and #665.
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Ctests were run on Acorn. I do not currently have rstprod access on the system, so only the HAFS and RTMA tests passed.
Checklist