-
Notifications
You must be signed in to change notification settings - Fork 153
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
update fix submodule for GFS v16.3.12 and soil analysis changes #695
update fix submodule for GFS v16.3.12 and soil analysis changes #695
Conversation
WCOSS2 ctests
Confirmed that the contrl and updat |
Hera ctests
A check of the updat and contrl wall times shows timings to vary within expected ranges
This is not a fatal fail. The
A check of updat and contrl wall times shows variability within normally observed ranges
This is not a fatal fail.
A check of updat and contrl wall times
does not reveal anomalous behavior. This is not a fatal fail. |
Hercules ctest
The
This message is cryptic. Output files
Differences are limited to the u and v wind components. The
However, this time it is the delp fields which differ
Rerun
Rerun of
This behavior is odd. Since the updat and contrl The
|
Orion ctest
The
A check of the
This points to Orion system issues. This is not a fatal fail. The wall clock limit for the |
@ClaraDraper-NOAA , this PR updates the GSI PR's need two peer reviews. I added you as a peer reviewer to this PR since you created the soil analysis fix files. Once this PR has two peer approvals, we can get the updated |
Orion ctests rerun
The
A check of the
Given run time variability on Orion this is not a fatal fail. |
@ClaraDraper-NOAA , all ctests yield acceptable results. This PR can be scheduled for merger pending your review and approval. |
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.
This includes both @ADCollard and my changes / additions to the fix files. Looks good to me.
@CoryMartin-NOAA , would you mind reviewing and approving this PR? I can not approve since I created the PR. This PR updates the |
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, thanks
@RussTreadon-NOAA should I handle the merge or would you like to? |
@CoryMartin-NOAA , I'm fine with shepherding this PR through the last gate. You already did something much more important. You opened the gate with your approval. Thanks! |
Description
GFS v16.3.12 updated ascii GSI fix files. GFS v16 soil analysis updates two ascii GSI fix files. This PR changes the fix submodule hash to bring in the updated ascii GSI fix files.
Fixes #640
Type of change
How Has This Been Tested?
Run ctests on WCOSS2, Hera, Orion, and Hercules with expected results obtained.
Checklist