Skip to content

Fix file changes for Saildrone, Windborne and GMI#38

Merged
RussTreadon-NOAA merged 10 commits into
NOAA-EMC:developfrom
ADCollard:develop
May 16, 2025
Merged

Fix file changes for Saildrone, Windborne and GMI#38
RussTreadon-NOAA merged 10 commits into
NOAA-EMC:developfrom
ADCollard:develop

Conversation

@ADCollard
Copy link
Copy Markdown
Collaborator

This PR complements GSI PR #878 and includes the changes to fix files required to assimilate Saildrone and Windborne observations. This addresses GSI Issue #849.

Also included are the changes required to turn on GMI observations GSI PR#692.

@ADCollard
Copy link
Copy Markdown
Collaborator Author

Thanks @RussTreadon-NOAA. I am on leave today and tomorrow but can check in occasionally.

Please add @xincjin-NOAA to review the GMI changes.

@emilyhcliu
Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA Could you add @xincjin-NOAA to reivew the changes for GMI? Thank you.

Copy link
Copy Markdown

@xincjin-NOAA xincjin-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 in two files (GMI): cloudy_radiance_info.txt global_satinfo.txt look good to me.

Thanks,

Copy link
Copy Markdown

@nicholasesposito nicholasesposito 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 but I'll defer to the others to approve since I'm unable to verify these numbers.

@ilianagenkova
Copy link
Copy Markdown
Contributor

@ADCollard , I trust the code generates reasonable stats for 301/401, 302/402?

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.

Approve.

@RussTreadon-NOAA RussTreadon-NOAA merged commit cb7fe0e into NOAA-EMC:develop May 16, 2025
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.

6 participants