Skip to content
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

Feature/nick e gpsro bufr #712

Merged
merged 43 commits into from
Nov 14, 2023
Merged

Feature/nick e gpsro bufr #712

merged 43 commits into from
Nov 14, 2023

Conversation

nicholasesposito
Copy link
Collaborator

This adds the GPS-RO bufr data converter (using the python api) to the GDASApp.

The two files added are:
~/ush/ioda/bufr2ioda/bufr2ioda_gpsro_bufr_combined.py
~/parm/ioda/bufr2ioda/bufr2ioda_gpsro_bufr_combined.json

This is validated in T20 issue #68

@nicholasesposito nicholasesposito self-assigned this Nov 8, 2023
@nicholasesposito nicholasesposito marked this pull request as draft November 8, 2023 16:21
Copy link
Contributor

@CoryMartin-NOAA CoryMartin-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor question, otherwise it looks good to me

@CoryMartin-NOAA CoryMartin-NOAA merged commit 16dbd25 into develop Nov 14, 2023
5 checks passed
@CoryMartin-NOAA CoryMartin-NOAA deleted the feature/NickE_gpsro_bufr branch November 14, 2023 21:23
CoryMartin-NOAA pushed a commit that referenced this pull request Nov 20, 2023
Updating GNSSRO converter and adding the yaml file to GDASApp end-to-end
testing based on [#712](#712).

New file:
parm/atm/obs/config/gnssro.yaml: Added QC filters in the YAML

Updated files:
parm/ioda/bufr2ioda/bufr2ioda_gnssro_bufr.json: Updated the satellite
information
ush/ioda/bufr2ioda/bufr2ioda_gnssro_bufr.py: Updated pccf, qfro, and
satelliteAscending flag

Refer to [issue #750](#750)
for testing details. JEDI/GSI comparisons revealed a cutoff near
obs=0.03 Rad due to super refraction 2. The number of observations after
QC in JEDI is also inconsistent with GSI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants