-
Notifications
You must be signed in to change notification settings - Fork 156
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 GPSRO obs error and add additional data #825
base: develop
Are you sure you want to change the base?
Conversation
@RussTreadon-NOAA : Could you please add @ADCollard , @emilyhcliu , and @LidiaCucurull-NOAA as reviewers? Thank you! |
@XuanliLi-NOAA , Lidia can not, at present, be assigned as a reviewer. She does not belong to NOAA-EMC. I sent her an invitation to be an outside collaborator with read permission. If you have not yet done so, please reach out to Lidia and share with her your request that she review this PR. |
Thank you so much, Russ. I'll send an email to request Lidia's review on this PR. |
WCOSS2 ctest results
The
|
Let me double check the qc section, it should be about the same. |
This is expected, I compared the numbers with those from my full resolution experiments. They are comparable. Penalty is smaller because of the larger obs error. |
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 changes look good. Please, double check the type of processing for GRACE-FO and Sentinel-6 in read_gps.f90. I believe these missions are not processed by UCAR/CDAAC, but unless I am reading it wrong, they are under the CDAAC group.
Also, you might want to consider removing new QC=7 since MetOP data is no longer rejected (and biases for future EPS missions will no longer exist either). Finally, the header text in setupbend.f90 seems to suggest that GRACE-FO is only assimilated below 8 km (the "< 8km" should be removed.)
@LidiaCucurull-NOAA , I just want to confirm, should we switch GRACE-FO and Sentinel-6 from the pccf to qfro check? Or should we remove them from both checks? They should have a quality flag. |
Xuanli and I met today. We discussed the code changes and some of my previous comments. I will check on which are the processing center for satellites ids 803, 804 and 66 to identify the right QC process to follow in read_gps. Also, it appears that the variable "repe_gps" is no longer used in setupbend.f90 with the new hybrid error model. Xuanli will test the code with the removal of this variable (which currently needs to be computed for each individual observation within a profile). This would save computational time and would make the code cleaner. |
Description
This update includes the following for GPSRO:
Replacing the current GPSRO observation error with the new hybrid three-cornered hat error model.
Allowing Grace-FO (SAID=803 & 804) into GSI.
Turning off the previous MetOp 8-km check.
Adding more QC categories.
More description and the results from the regression tests are included in the issue #819
Resolves #819
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Checklist