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

Add an option in dynamics to calculate omega in non-hydrostatic runs similar to hydrostatic method #2327

Merged
merged 19 commits into from
Jul 3, 2024

Conversation

XiaqiongZhou-NOAA
Copy link
Contributor

@XiaqiongZhou-NOAA XiaqiongZhou-NOAA commented Jun 14, 2024

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 is to bring an update in dycore. An option is added in dynamics to calculate the omega in non-hydrostatics runs similar to hydrostatic method.

Commit Message:

    * atmos_cubed_sphere - 
                         A new parameter, pass_full_omega_to_physics_in_non_hydrostatic_mode, has been added in 
                         dycore. When set to true, the omega calculation will match that of hydrostatic runs, using the 
                         formula omega = dp/dt. The default value is false, where the calculation remains
                         omega = w * dp/dz as before.

Priority:

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
    • Jet
    • Gaea
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

@zach1221
Copy link
Collaborator

@XiaqiongZhou-NOAA would you be able to sync up your PR branches and provide Hera regression testing log for this PR?

@jkbk2004 jkbk2004 added the No Baseline Change No Baseline Change label Jun 27, 2024
@jkbk2004 jkbk2004 added the Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. label Jun 28, 2024
@jkbk2004
Copy link
Collaborator

@zach1221 @FernandoAndrade-NOAA @BrianCurtis-NOAA This PR is ready.

@FernandoAndrade-NOAA
Copy link
Collaborator

Please be aware that there may be issues with the Jet RTs, there has been a reduction in performance due to hardware failures. The Jet support team is working on getting those resolved. I have a run going right now, but it is extremely slow. @jkbk2004 FYI

@BrianCurtis-NOAA
Copy link
Collaborator

Hercules logs show two failed tests due to timeout

@zach1221
Copy link
Collaborator

zach1221 commented Jul 2, 2024

Hercules logs show two failed tests due to timeout

Yes, we think they are failing due to spack issue with mvapich2. It shows as timeout though in the rt log.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Jul 2, 2024

Can we move on for merging? It's better to catch up subcomponent PR to be merged. I mean due to July 4 schedule.

@zach1221 zach1221 merged commit 8e7b61b into ufs-community:develop Jul 3, 2024
3 checks passed
WalterKolczynski-NOAA pushed a commit to NOAA-EMC/global-workflow that referenced this pull request Aug 23, 2024
Add a parameter "pass_full_omega_to_physics_in_non_hydrostatic_mode" in
FV3 namelist. It was set to "true" to use a new method to diagnose
omega. This PR is based on the /ufs-community/ufs-weather-model#2327)

Corresponding parameter changed in GFSv17 related regression tests
ufs-community/ufs-weather-model#2373))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Baseline Change No Baseline Change Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

control_ras fails to compare on Acorn
5 participants