Skip to content

Conversation

@BoYang-NOAA
Copy link

@BoYang-NOAA BoYang-NOAA commented Apr 22, 2025

Commit Queue Requirements:

  • Fill out all sections of this template.
  • All sub component pull requests have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
  • Commit 'test_changes.list' from previous step

Description:

Add ECMWF Non-stationary gravity wave scheme in UGWDV1 gravity wave suite. If do_ngw_ec is set to true, use the ECMWF NGWD scheme. Conversely, if do_ngw_ec is false, use the original UGWDv1 NGWD scheme. do_ngw_ec is set to false by default.

Commit Message:

* UFSWM - 
 *FV3
    * ccpp-physics - 
  

Priority:

  • Normal

Git Tracking

UFSWM:

  • None

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • None

Changes

Regression Test Changes (Please commit test_changes.list):

  • No Baseline Changes.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • No Updates

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • GaeaC5
    • GaeaC6
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

@BoYang-NOAA BoYang-NOAA marked this pull request as ready for review April 22, 2025 16:58
@grantfirl
Copy link
Collaborator

@BoYang-NOAA I noticed that there is no regression test log added for the tests that you've done. Could you push the tests/logs/RegressionTests_*.log from the platform where you ran the tests please?

@grantfirl
Copy link
Collaborator

@BoYang-NOAA Please fill out something for the commit messages in the PR description as well.

@BoYang-NOAA
Copy link
Author

BoYang-NOAA commented Apr 30, 2025 via email

@grantfirl
Copy link
Collaborator

@BoYang-NOAA This has been combined into #2790. Let's leave this PR open until the combined PR is merged in case it runs into problems.

@jkbk2004
Copy link
Collaborator

merged with #2790

@jkbk2004 jkbk2004 closed this Jul 11, 2025
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