Skip to content

Fix tropical cyclone HSD case#3159

Open
ulmononian wants to merge 1 commit intoufs-community:developfrom
ulmononian:feature/fix_tc
Open

Fix tropical cyclone HSD case#3159
ulmononian wants to merge 1 commit intoufs-community:developfrom
ulmononian:feature/fix_tc

Conversation

@ulmononian
Copy link
Collaborator

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.
  • x] Fill out all sections of this template.

Description:

resolve timeout issues with tropical cyclone hsd case (reported in #3092) by utilizing default fd_ufs.yaml rather than original version (fd_ufs.tc.yaml) located in INPUTDATAROOT/.../HSD_input_data/tropical_cyclone/fix

Commit Message:

switch tc hsd case to default fd_ufs.yaml

* UFSWM - 
  * AQM - 
  * CDEPS - 
  * CICE - 
  * CMEPS - 
  * CMakeModules - 
  * UFSATM - 
    * ccpp-physics - 
    * atmos_cubed_sphere - 
  * GOCART - 
  * HYCOM - 
  * MOM6 - 
  * NOAHMP - 
  * WW3 - 
  * fire_behavior
  * stochastic_physics - 

Priority:

  • Critical Bugfix: Reason
  • High: Reason
  • Normal

Git Tracking

UFSWM:

Sub component Pull Requests:

  • None

UFSWM Blocking Dependencies:

  • Blocked by #
  • [ x] 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.
  • [x ] No Baseline Changes.

Input data Changes:

  • [x ] None.
  • PR adds input data.
  • PR changes existing input data.

Library Changes/Upgrades:

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

Testing Log:

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

@ulmononian
Copy link
Collaborator Author

i ran this on ursa for FHMAX=3 (default for TC case) using ./ufs_test.sh -a epic -k -r -c -n"tropical_cyclone intel" and the model ran to completion as expected (out.txt). hopefully this out log is enough to get this fix in; don't think it is worth all the comp. resources to run the full RT suite for this simple fix that only impacts a single hsd case inside tests-dev..

@gspetro-NOAA fyi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Evaluating

Development

Successfully merging this pull request may close these issues.

Tropical cyclone HSD fails in develop branch

1 participant