-
Notifications
You must be signed in to change notification settings - Fork 35
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
Add observation YAMLs for AMSU-A, ATMS, SSMIS, SEVIRI, and AVHRR #360
Conversation
@emilyhcliu Apparently the develop version was not pointing to the right UFO build after all. I just pushed a commit to update
|
@emilyhcliu |
@emilyhcliu I'll work on the final AMSU-A/ATMS QC in the morning. There will not be exact agreement, but we should be screening out obs with missing values anyway. We don't want to be pointing to a version of CRTM that is not under change control. |
.... actually I am going to see if I can make a modified CRTM tag.... |
…ation. Therefore, change MetaData/heightOfSurface to GeoVaLs/surface_geopotential_height
…DASApp into feature/yaml4radiance
Yes |
With the latest changes, I get the relevant tests to pass on Hera. I will look in detail at the summary output plots and will also test on Orion:
|
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.
Did not test, but assuming it will be all good once @ADCollard finishes his tests. Thanks a lot @emilyhcliu for all of your hard work on this and getting it in GDASApp!
This IODA PR broke the eva scripts which meant I could not produce the output plots. A one line change to `ush/eva/gen_eva_obs.yaml' fixes this and was pushed here. |
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.
@CoryMartin-NOAA I approve this request but to be doubly sure (as I have been making multiple changes to this PR myself) can you do a clean checkout and build of this branch and run ush/ufoeval/test_yamls.sh
as in this comment?
Note that there should be no need to change any files as long as the GDASApp
and machine
environment variables are set correctly.
note that I also had to set
I'm impatient and going home now but I think this is sufficient to say it is ready to merge, thus I will do so :-) |
Add observation YAMLs for AMSU-A, ATMS, SSMIS, SEVIRI, and AVHRR
Pass benchmarks are provided in the YAMLs for unit tests.
Please see the following JEDI-T2O issues for documentation and validation results:
JEDI-T2O#42 --- AMSU-A (100% match)
JEDI-T2O#44 --- ATMS (Difference: 1 point for NPP and 5 points for N20)over south sea ice area from hydrometer check)
JEDI-T2O#55 --- SSMIS with Azadeh (100% match)
JEDI-T2O#53 --- AVHRR with Azadeh ( Difference: 1 point for N19)
JEDI-T2O#52 --- SEVIRI with Xuanli (100% match)
Summary plots for validation will be posted later.