Skip to content

Conversation

@rtodling
Copy link
Contributor

@rtodling rtodling commented Feb 9, 2026

Bring back aerosol-affected radiance assimilation to GSI.

This has been tested and it's zero-diff when not used. Works as expected for say, IASI when setting iaero = 1 for channels in the range of 8.26-15.5 micros (645 - 1210 cm^-1).

There are some puzzles, though:

1 - triggering some of AIRS channels in the range of 3-5 and 8-13 microns (indexes 435-1285; and 1865-2377) don't seem to change the results.

2 - to fully exercise this properly, the user has to carefully trigger the radiance channels (IR) affected by aerosols, explicitly in satinfo.

3 - a corresponding feature has been added to GEOS-IT v5.29.4.7 and is being the aerosol affected feature is being tested on volcanic eruption case. That work should help us determine a more complete set of channel-specific trigggers for variious IR instruments: AIRS, AVHRR, SEVIRI and HIRS - beyond the IASI channels already included in the changes here.

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Label error. Requires at least 1 of: 0 diff, 0 diff trivial, Non 0-diff, 0 diff structural, 0-diff trivial, Not 0-diff, 0-diff, automatic, 0-diff uncoupled. Found:

mathomp4
mathomp4 previously approved these changes Feb 9, 2026
@mathomp4
Copy link
Member

mathomp4 commented Feb 9, 2026

Oh whoops. Didn't see this was in Draft form.

Welp, it's pre-approved! 😄

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Label error. Requires at least 1 of: 0 diff, 0 diff trivial, Non 0-diff, 0 diff structural, 0-diff trivial, Not 0-diff, 0-diff, automatic, 0-diff uncoupled. Found:

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Label error. Requires at least 1 of: 0 diff, 0 diff trivial, Non 0-diff, 0 diff structural, 0-diff trivial, Not 0-diff, 0-diff, automatic, 0-diff uncoupled. Found:

@rtodling rtodling added enhancement New feature or request 0 diff The changes in this pull request have verified to be zero-diff with the target branch. labels Feb 10, 2026
@rtodling rtodling marked this pull request as ready for review February 10, 2026 16:41
@rtodling rtodling requested review from a team as code owners February 10, 2026 16:41
@rtodling rtodling requested review from mathomp4 and removed request for mathomp4 February 10, 2026 16:41
@rtodling rtodling merged commit c68f5a1 into develop Feb 11, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch. enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants