-
Notifications
You must be signed in to change notification settings - Fork 39
Completes LETKF task, adds WCDA CI test #1287
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
Conversation
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.
👍
Co-authored-by: Cory Martin <cory.r.martin@noaa.gov>
Co-authored-by: Cory Martin <cory.r.martin@noaa.gov>
Adds run and finalize methods to marine LETKF task, experiment yaml for gw-ci in GDASApp, workflow additions, removes bugs found on the way, and completes the bulk of the work on LETKF task. Conversion of fields to increments pending. Partially resolves NOAA-EMC/GDASApp#1091 and NOAA-EMC/GDASApp#1251 Mutual dependency with GDASApp PR NOAA-EMC/GDASApp#1287 and IC fix file issue #2944 (comment) --------- Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov> Co-authored-by: David Huber <69919478+DavidHuber-NOAA@users.noreply.github.com>
The global-workflow PR has been merged: NOAA-EMC/global-workflow#2944 |
Automated Global-Workflow GDASApp Testing Results:
Build: SUCCESS
|
Are any of those failures unexpected? |
@AndrewEichmann-NOAA , the only failures I expect are
I think @apchoiCMD is investigating these failures, |
@givelberg is working on fixing the obs ctest, check the log for the rest. I suspect the jobs timed out. |
@AndrewEichmann-NOAA , can you fix the conflict and turn off the hybrid test? It won't work until a few things are merged in the g-w. |
@guillaumevernieres Done |
Automated GW GDASApp Testing Results:
|
Completes LETKF task, adds WCDA CI test
Mutually dependent on NOAA-EMC/global-workflow#2944