Skip to content

Add extra adjustment for noaa21#122

Merged
ninahakansson merged 4 commits intofoua-pps:mainfrom
ninahakansson:vgac_noaa21_extra_sbafs
Jan 28, 2026
Merged

Add extra adjustment for noaa21#122
ninahakansson merged 4 commits intofoua-pps:mainfrom
ninahakansson:vgac_noaa21_extra_sbafs

Conversation

@ninahakansson
Copy link
Collaborator

@ninahakansson ninahakansson commented Dec 17, 2025

Add extra adjustment for NOAA21 before the NN-SBAFs.

Adjust NOAA21 to VIIRS, save as VIIRS:
vgac2pps.py VGAC_VN2102MOD_A2023365_0003_n05898_K005.nc --noaa21_to_snpp v1 (not possible any longer)

Adjust NOAA21 to VIIRS, than apply NN-SBAF and save as AVHRR:
vgac2pps.py VGAC_VN2102MOD_A2023365_0003_n05898_K005.nc --noaa21_to_snpp v1 -n19 NN_v4

  • Closes #xxxx <!-- remove if there is no corresponding issue
  • Tests added
  • Tests passed: Passes pytest level1c4pps
  • Passes flake8
  • Add your name to AUTHORS.md if not there already

@coveralls
Copy link

coveralls commented Dec 17, 2025

Pull Request Test Coverage Report for Build 21395415264

Details

  • 11 of 17 (64.71%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.03%) to 75.194%

Changes Missing Coverage Covered Lines Changed/Added Lines %
level1c4pps/vgac2pps_lib.py 11 17 64.71%
Totals Coverage Status
Change from base Build 19823395775: -0.03%
Covered Lines: 1849
Relevant Lines: 2405

💛 - Coveralls

@ninahakansson ninahakansson marked this pull request as ready for review January 13, 2026 11:32
Copy link
Contributor

@BengtRydberg BengtRydberg left a comment

Choose a reason for hiding this comment

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

Looks godd. Added two minor suggestions for readability.

@ninahakansson ninahakansson merged commit b7339b6 into foua-pps:main Jan 28, 2026
8 checks passed
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.

3 participants