Skip to content

Feature/amsr2 in allsky updates#41

Merged
RussTreadon-NOAA merged 4 commits into
NOAA-EMC:developfrom
jianjunj:feature/amsr2_in_allsky_updates
Jun 9, 2025
Merged

Feature/amsr2 in allsky updates#41
RussTreadon-NOAA merged 4 commits into
NOAA-EMC:developfrom
jianjunj:feature/amsr2_in_allsky_updates

Conversation

@jianjunj
Copy link
Copy Markdown
Contributor

@jianjunj jianjunj commented Jun 6, 2025

Add the information of ASMR2 in global_scaninfo.txt, global_satinfo.txt, and cloudy_radiance_info.txt. Set channels 7-13 as active. Observational bias data are taken from GEOS. These settings are preliminary and need to be tuned with GFS.

@jianjunj
Copy link
Copy Markdown
Contributor Author

jianjunj commented Jun 6, 2025

@RussTreadon-NOAA Can I request Andrew, Emily and Haixia to review this PR? Thank you.

Copy link
Copy Markdown
Collaborator

@ADCollard ADCollard 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

Comment thread global_satinfo.txt
@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor

@jianjunj , for this PR to move forward

  1. confirm confirmation from @emilyhcliu that you addressed her concerns
  2. resolve the conversation upon her confirmation
  3. request a re-review from @emilyhcliu

This PR can be merged into GSI-fix develop upon @emilyhcliu 's re-review and approval or @HaixiaLiu-NOAA 's review an approval.

@RussTreadon-NOAA RussTreadon-NOAA self-requested a review June 9, 2025 14:44
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.

Proposed changes have two peer reviews and approvals.

Approve.

@RussTreadon-NOAA RussTreadon-NOAA merged commit 37b1872 into NOAA-EMC:develop Jun 9, 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.

4 participants