Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.9.1 fix: Update subsetting procedure in run_prdgen task #354

Merged
merged 1 commit into from
Apr 29, 2024
Merged

v0.9.1 fix: Update subsetting procedure in run_prdgen task #354

merged 1 commit into from
Apr 29, 2024

Conversation

BenjaminBlake-NOAA
Copy link
Contributor

DESCRIPTION OF CHANGES:

  • 250 mb UGRD and VGRD have been missing from the CONUS and Alaska pressure level grib2 output files since the update to v0.9.1 on April 18th. This issue was discovered due to the RRFS Python graphics jobs not finding 250 mb UGRD in the CONUS grib2 files. With the v0.9.1 update, 91 records were removed from the prslev files. This caused 250 mb UGRD and 250 mb VGRD to be split up onto two separate files (record 250, the last record in one file, and record 251, the first record in another file). The -submsg_uv option is used with wgrib2, and this caused an issue when UGRD and VGRD were not grouped together. For creating the CONUS and Alaska grib2 files, subtask 1 now processes records 1-251 instead of 1-250, and subtask 2 now processes records 252-500 instead of 251-500.
  • If additional products are added/removed to the RRFS grib2 files in the future, we will need to check and make sure the run_prdgen subsetting is still working correctly.
  • This change should also be added to the production/RRFS.v1 branch.

TESTS CONDUCTED:

Tested in the v0.9.1 real-time parallel.

Machines/Platforms:

  • WCOSS2
    • Cactus/Dogwood
    • Acorn
  • RDHPCS
    • Hera
    • Jet
    • Orion
    • Hercules

Test cases:

  • Engineering tests
    • Non-DA engineering test
    • DA engineering test
      • Retro
      • Ensemble
      • Parallel
  • RRFS fire weather
  • RRFS_A:
  • RRFS_B:
  • RTMA:
  • Others:

CONTRIBUTORS (optional):

@MatthewPyle-NOAA suggested the issue could be related to the vector components being split up.

Copy link
Contributor

@MatthewPyle-NOAA MatthewPyle-NOAA left a comment

Choose a reason for hiding this comment

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

Looks like one to get in quickly!

@MatthewPyle-NOAA MatthewPyle-NOAA merged commit c5220fd into NOAA-EMC:dev-sci Apr 29, 2024
@BenjaminBlake-NOAA BenjaminBlake-NOAA deleted the feature/prdgen branch April 30, 2024 11:21
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.

2 participants