Skip to content

Conversation

@XiaqiongZhou-NOAA
Copy link
Contributor

@XiaqiongZhou-NOAA XiaqiongZhou-NOAA commented Jan 23, 2026

Commit Queue Requirements:

  • This PR addresses a relevant WM issue (if not, create an issue).
  • All subcomponent pull requests (if any) have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines), preferably on Ursa (Derecho or Hercules are acceptable alternatives). Exceptions: documentation-only PRs, CI-only PRs, etc.
    • Commit log file w/full results from RT suite run (if applicable).
    • Verify that test_changes.list indicates which tests, if any, are changed by this PR. Commit test_changes.list, even if it is empty.
  • Fill out all sections of this template.

Description:

Update ccpp_physics to
modify 'dtfac' OGWD tendency limiter in drag_suite_psl to match that in drag_suite_run.

Commit Message:

* UFSWM - modify 'dtfac' OGWD tendency limiter in drag_suite_psl to match that in drag_suite_run.
    * UFSATM - modify 'dtfac' OGWD tendency limiter in drag_suite_psl to match that in drag_suite_run.
        * ccpp-physics - modify 'dtfac' OGWD tendency limiter in drag_suite_psl to match that in drag_suite_run.

Priority:

  • Critical Bugfix: Reason
  • High: Fix model instability issue
  • Normal

Git Tracking

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • Blocked by #
  • None

Documentation:

  • Documentation update required.
    • Relevant updates are included with this PR.
    • A WM issue has been opened to track the need for a documentation update; a person responsible for submitting the update has been assigned to the issue (link issue).
  • Documentation update NOT required.
    • Explanation:

Changes

Regression Test Changes (Please commit test_changes.list):

  • PR Adds New Tests/Baselines.
  • PR Updates/Changes Baselines.
  • No Baseline Changes.

Input data Changes:

  • None.
  • PR adds input data.
  • PR changes existing input data.

Library Changes/Upgrades:

  • Required
    • Library names w/versions:
    • Git Stack Issue (JCSDA/spack-stack#)
  • No Updates

Testing Log:

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

@github-project-automation github-project-automation bot moved this to Evaluating in PRs to Process Jan 23, 2026
@XiaqiongZhou-NOAA XiaqiongZhou-NOAA marked this pull request as ready for review January 23, 2026 15:41
@gspetro-NOAA gspetro-NOAA added Baseline Updates Current baselines will be updated. Priority: High labels Jan 23, 2026
@gspetro-NOAA
Copy link
Collaborator

@XiaqiongZhou-NOAA In order to schedule this PR for the commit queue, we'll need you to run the full RT suite (preferably on Ursa, although Hercules or another platform would do) and push the logs and the test_changes.list file showing which baselines will change. The Commit Message section of the template also needs to be filled out with a commit message for each subcomponent.

@gspetro-NOAA gspetro-NOAA moved this from Evaluating to Pre-testing required in PRs to Process Jan 24, 2026
@gspetro-NOAA gspetro-NOAA added UFSATM There are changes to the UFSATM repository. CCPP There are changes to a CCPP repository. labels Jan 24, 2026
@XiaqiongZhou-NOAA
Copy link
Contributor Author

XiaqiongZhou-NOAA commented Jan 26, 2026

test_changes.list is added. New baseline on URSA: /scratch4/NCEPDEV/stmp/Xiaqiong.Zhou/RT_BASELINE/Xiaqiong.Zhou/FV3_RT/REGRESSION_TEST
Just wondering which rt.sh flags should be used to generate log file to be committed for full regression tests.

@dpsarmie
Copy link
Collaborator

The logs to add to the PR should be in the ufswm directory that you used to run the RTs. ex. ./ufs-weather-model/tests/logs/RegressionTests_ursa.log, substitute the log for the machine used in testing.

There shouldn't be any special flags needed in rt.sh.

@XiaqiongZhou-NOAA
Copy link
Contributor Author

The logs to add to the PR should be in the ufswm directory that you used to run the RTs. ex. ./ufs-weather-model/tests/logs/RegressionTests_ursa.log, substitute the log for the machine used in testing.

There shouldn't be any special flags needed in rt.sh.

Daniel: Doe you mean the log files that indicates fails for all RTs with baseline changed?

@dpsarmie
Copy link
Collaborator

Daniel: Doe you mean the log files that indicates fails for all RTs with baseline changed?

Yes, that log file with the failures listed.

@gspetro-NOAA gspetro-NOAA moved this from Pre-testing required to Waiting for Reviews (subcomponent) in PRs to Process Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Baseline Updates Current baselines will be updated. CCPP There are changes to a CCPP repository. Priority: High UFSATM There are changes to the UFSATM repository.

Projects

Status: Waiting for Reviews (subcomponent)

Development

Successfully merging this pull request may close these issues.

3 participants