-
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
Changes prepoceanobs to using newly refactored BUFR converters #1352
Conversation
Install this GDASApp PR and g-w PR #2992 on Hera. Run
Each of these jobs fail with the error message
Do we have a fix for these failures in the development pipeline? |
Thanks for pointing this out @RussTreadon-NOAA . I created an issue off of your comment above. |
Automated GW GDASApp Testing Results:
|
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.
The ci failure isn't related to this PR, I forgot that the b-matrix pr was merged and develop will be broken until the g-w pr is merged as well.
I'm merging this keeping in mind (right ...) that we might have to revert your changes to the obs list yaml.
Just like the title. This mainly changes the converters used to the new ones that use yaml files, the templates for which are included, along with a little script that does most of the work of converting the earlier JSON templates to yaml. Also removes from
ocean.py
some methods for plotting, but can't go into global-workflow because of lack of the necessary python modules in that environment.