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

Fix to gfs_bufr code and job #879

Conversation

KateFriedman-NOAA
Copy link
Member

Description

This PR commits changes made and tested on WCOSS2 by NCO, GDIT, and @BoCui-NOAA to resolve job failures in the gfs_atmos_postsnd job in operations. A small memory adjustment for the fbwind job is also included, along with an update to a transfer list file.

Updates related to gfs_bufr are:

  • Correction to gfs_bufr code (by @BoCui-NOAA )
  • Change to gfs_bufr build to no longer build with -qopenmp.
  • Adjustments to job resources based on testing by @BoCui-NOAA, @WeiWei-NCO , and GDIT.

Thanks to @BoCui-NOAA , GDIT, @HuiyaChuang-NOAA , and @WeiWei-NCO for troubleshooting!

This update will be packaged with the recent gfs_forecast job update (PR #874). It will result in an RFC in WCOSS2 operations and a version increment to GFSv16.2.1. New release notes are also included.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • This change requires a documentation update

How Has This Been Tested?

NCO, GDIT, and @BoCui-NOAA tested these updates in the para GFS on WCOSS2.

Refs #399

- Resource adjustments for the postsnd bufr_sounding job (based on GDIT
and Bo Cui testing).
- Fix to gfs_bufr code (Bo Cui).
- gfs_bufr build update to no longer build with "-qopenmp".

Refs: NOAA-EMC#399
@KateFriedman-NOAA KateFriedman-NOAA added the production update Processing update in production label Jun 28, 2022
@KateFriedman-NOAA KateFriedman-NOAA added this to the WCOSS2 - GFSv16.2.0 milestone Jun 28, 2022
@KateFriedman-NOAA KateFriedman-NOAA self-assigned this Jun 28, 2022
Copy link
Contributor

@aerorahul aerorahul 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.
These changes were already part of para, so they are well tested.

Copy link

@WeiWei-NCO WeiWei-NCO 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. Thanks!

@KateFriedman-NOAA KateFriedman-NOAA merged commit 41c97a5 into NOAA-EMC:feature/ops-wcoss2 Jun 28, 2022
@BoCui-NOAA
Copy link
Contributor

Thanks for all your quick action. @KateFriedman-NOAA @WeiWei-NCO and @aerorahul

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
production update Processing update in production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants