-
Notifications
You must be signed in to change notification settings - Fork 31
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 JEDI hashes to include Model Variable Renaming Sprint #1355
Conversation
Automated Global-Workflow GDASApp Testing Results:
|
Automated GDASApp Testing Results:
|
Automated GDASApp Testing Results:
|
Automated GDASApp Testing Results:
|
Will keep this PR in draft mode until GDASApp and jcb-gdas yamls and templates work with updated JEDI hashes. |
The changes in this PR have been tested in g-w PR #2992. Tests completed include:
|
This PR SHOULD NOT be merged into |
@guillaumevernieres , @AndrewEichmann-NOAA , @DavidNew-NOAA, @CoryMartin-NOAA - this PR is ready for final review. It's been tested in g-w #2992. |
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, especially given all the testing. Nice work, @RussTreadon-NOAA
Thank you @DavidNew-NOAA and @CoryMartin-NOAA for the quick approvals. I'll go ahead and merge. |
This PR updates JEDI hashes in
sorc/
to bring in changes associated with the Model Variable Renaming Sprint. This is required to fully resolve #1313.Resolves #1313
Resolves #1362
Depends: jcb-gdas #45