Skip to content

Adding VIIRS-M Radiance Data Assimilation in GEOS - #237

Open
ahrem1234 wants to merge 2 commits into
developfrom
feature/alee_viirs
Open

Adding VIIRS-M Radiance Data Assimilation in GEOS#237
ahrem1234 wants to merge 2 commits into
developfrom
feature/alee_viirs

Conversation

@ahrem1234

@ahrem1234 ahrem1234 commented May 22, 2026

Copy link
Copy Markdown
Collaborator

This PR is primarily for adding VIIRS radiance observation in GEOS.
VIIRS radiance data for channels M12-16 are read, and that for channels M12, M15, and M16 are assimilated.

This PR is to achieve the following purposes:

  1. add read_viirs.f90
  2. correct calcsun module
  3. correct surface states (e.g. skin temperature) reading - needs further investigation

+) only use water surface for BC coeff. update

@ahrem1234
ahrem1234 requested review from a team as code owners May 22, 2026 16:04
@github-actions

Copy link
Copy Markdown

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:

1 similar comment
@github-actions

Copy link
Copy Markdown

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
Copy Markdown

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:

@ahrem1234 ahrem1234 added Non 0-diff The changes in this pull request are non-zero-diff enhancement New feature or request labels Jul 21, 2026
@rtodling

Copy link
Copy Markdown
Contributor

I am looking to see if I approve this, but what I think this is missing is some type of result - even if from a single analysis test. Can something be provided here. Like, perhaps what kind of increments we get when only VIIRS is assimilated vs perhaps avhrr3_metop-b (or something)?

@rtodling

Copy link
Copy Markdown
Contributor

@ahrem1234 you need to also edit the CHANGE_LOG file and add a note there about this change otherwise GIT checks will continue to fail.

@ahrem1234

ahrem1234 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator Author

Thank you for your comments, Ricardo. I have attached some forecast scorecards and radman plots that I got from comparing CTL(x0050) to EXP(adding VIIRS on NPP and NOAA-20). I hope this helps clarify the impact of this new VIIRS radiance DA.
VIIRS_results.pptx

@ahrem1234

Copy link
Copy Markdown
Collaborator Author

@ahrem1234 you need to also edit the CHANGE_LOG file and add a note there about this change otherwise GIT checks will continue to fail.

Could you point out where I can find the CHANGE_LOG file, so that I can add the note, please? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Non 0-diff The changes in this pull request are non-zero-diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants