Skip to content

Add NOAA-21 omps obs and tidy up convinfo#39

Merged
RussTreadon-NOAA merged 12 commits into
NOAA-EMC:developfrom
ADCollard:develop
May 23, 2025
Merged

Add NOAA-21 omps obs and tidy up convinfo#39
RussTreadon-NOAA merged 12 commits into
NOAA-EMC:developfrom
ADCollard:develop

Conversation

@ADCollard
Copy link
Copy Markdown
Collaborator

This PR complements NOAA-EMC/GSI#878 and follows on from #38. This includes the changes to fix files required to assimilate NOAA-21 OMPS nadir profile and total column observations. It also removes references to obs types 180/280 with subtype 02. This refers to saildrone observations in prepbufr that have been moved to their own obstype (302/402).

@ADCollard
Copy link
Copy Markdown
Collaborator Author

@HaixiaLiu-NOAA, @ilianagenkova and @dmerkova, could you take a look, please?

Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA RussTreadon-NOAA left a comment

Choose a reason for hiding this comment

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

Looks good to me. Can merge pending peer review & approval.

Approve.

Copy link
Copy Markdown

@dmerkova dmerkova left a comment

Choose a reason for hiding this comment

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

windborn changes looks good to me

Copy link
Copy Markdown
Contributor

@HaixiaLiu-NOAA HaixiaLiu-NOAA left a comment

Choose a reason for hiding this comment

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

changes look good. Approved

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor

Thank you @dmerkova and @HaixiaLiu-NOAA for your reviews and approvals. Merging now.

@RussTreadon-NOAA RussTreadon-NOAA merged commit 2f2a3a0 into NOAA-EMC:develop May 23, 2025
@ADCollard
Copy link
Copy Markdown
Collaborator Author

Thank you @dmerkova @HaixiaLiu-NOAA @RussTreadon-NOAA !

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.

4 participants