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

Update inline post #2489

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Conversation

WenMeng-NOAA
Copy link
Contributor

@WenMeng-NOAA WenMeng-NOAA commented Nov 4, 2024

branch.

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:

This PR updates inline post for SFS and GEFS developments by:

  • Updating the upp submodule within FV3 with recent changes from the UPP repository

  • Updating inline post read interface

Commit Message:

* UFSWM - 
  * FV3 - 
    * upp - 

Priority:

  • High: Support both GEFS and SFS developments

Git Tracking

UFSWM:

Sub component Pull Requests:

  • AQM:
  • CDEPS:
  • CICE:
  • CMEPS:
  • CMakeModules:
  • FV3: NOAA-EMC/fv3atm#880
    • ccpp-physics:
    • atmos_cubed_sphere:
  • GOCART:
  • HYCOM:
  • MOM6:
  • NOAHMP:
  • WW3:
  • fire_behavior:
  • stochastic_physics:
  • None

UFSWM Blocking Dependencies:

  • None

Changes

Regression Test Changes (Please commit test_changes.list):

  • PR Updates/Changes Baselines.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • No Updates

Testing Log:

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

@@ -23,3 +23,4 @@ export CCPP_SUITE=FV3_RAP_clm_lake
export LKM=1
export IOPT_LAKE=2
export FRAC_ICE=.true.
export WRITE_DOPOST=.false.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jkbk2004 @ulmononian Can you also incorporate this change in PR #2461? With the recent UPP updates, the inline post is not working for this test anymore.

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.

Update of UPP into UFS weather model
1 participant