-
Notifications
You must be signed in to change notification settings - Fork 3
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
Modify twindow.rc, twindow_m2.rc to add (excluded) newer wind types #288
base: develop
Are you sure you want to change the base?
Conversation
about unknown wind types.
Label error. Requires at least 1 of: 0 diff, 0 diff trivial, Non 0-diff, 0 diff structural, 0-diff trivial, Not 0-diff, 0-diff, automatic, 0-diff uncoupled. Found: |
Tests with the modified rc files gave identical results in BUFR output. These messages no longer printed: |
@gmao-msienkie I believe these changes are mainly aimed at R21C (M21C), am I correct? It seems they would be ok going into develop - but these would need to be brought in to R21C. Can you confirm, pls? |
These changes are actually only relevant for the older systems like MERRA2 and FPIT. It is run as a data preprocessing step in operations. We wanted to limit the amount of wind data read in to avoid overflow of the internal array, and to keep the amount of AMVs assimilated similar to what had been using. |
Some new wind types (NC005072, NC005081) that are not used.
This will avoid messages in the output about these unknown wind types.
There should be no difference in the output.