-
Notifications
You must be signed in to change notification settings - Fork 111
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
gdas_init utility bug fix #912
gdas_init utility bug fix #912
Conversation
@CoryMartin-NOAA tested the branch on the case that was failing for him. It worked. |
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 @GeorgeGayno-NOAA !
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.
I gave this a try out on Jet and was able to create ICs. Everything looks good. Thanks!
Thanks, can you also try on 's4'? I don't have access to that machine. |
Sure, I'll let you know how that goes. |
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!
DESCRIPTION OF CHANGES:
When using warm restart files as input to the chgres step (v15/GDAS and v16 retro/GDAS options), it would fail because it was reading from the wrong input data orography directory. Update the scripts to point to the correct directory.
TESTS CONDUCTED:
Two test cases were done on Hera, Jet and WCOSS2.
Test 1
Test 2
Both tests completed normally and produce the required coldstart files.
The 's4' scripts were tested by @DavidHuber-NOAA.
DOCUMENTATION:
N/A
ISSUE:
Fixes #908.