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

Third reconciliation PR from production/RRFS.v1 #2482

Merged
merged 15 commits into from
Nov 25, 2024

Conversation

grantfirl
Copy link
Collaborator

@grantfirl grantfirl commented Oct 29, 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 mainly work from @AndersJensen-NOAA. This PR is identical to #2161, but targets develop instead of the RRFSv1 release branch.

This PR is the first of many that will modernized, modularize, and streamline Thompson-Eidhammer microphysics.

This PR includes:

The addition of parameterized kind: REAL -> real(kind_phys)
Consistent indentation
Removal of GOTO statements
Add physical constant metadata needed by Thompson MP

Commit Message:

* UFSWM - Thompson-Eidhammer microphysics code formatting
  * FV3 - Thompson-Eidhammer microphysics code formatting
    * ccpp-physics - Thompson-Eidhammer microphysics code formatting

Priority:

  • High: Needed for SRW App

Git Tracking

UFSWM:

  • None

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • None

Changes

Regression Test Changes (Please commit test_changes.list):

  • PR Updates/Changes Baselines.

All tests that use Thompson MP will have non bit-for-bit results due to high significant digit changes in physical constants. There is no substantial change in results as tested by @AndersJensen-NOAA in #2161.

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)

@grantfirl grantfirl marked this pull request as ready for review November 8, 2024 17:40
@jkbk2004
Copy link
Collaborator

@grantfirl can you sync up branch? we can start working on this pr.

@grantfirl
Copy link
Collaborator Author

@grantfirl can you sync up branch? we can start working on this pr.

@jkbk2004 OK, should be good to go.

@jkbk2004 jkbk2004 added Baseline Updates Current baselines will be updated. Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. labels Nov 21, 2024
@FernandoAndrade-NOAA
Copy link
Collaborator

Changes also occurred in intelllvm cases + control_c48_lnd_iau_intel across Hera, Gaea, and Jet on my side. The same changes are likely for Orion and Hercules as well. Recreating baselines for those cases now.

@jkbk2004 jkbk2004 merged commit 144ccb0 into ufs-community:develop Nov 25, 2024
4 checks passed
@BrianCurtis-NOAA
Copy link
Collaborator

I would like to reiterate my request that someone from EMC needs to review all PR's before merge. I was in the middle of my review when it was merged.

@jkbk2004
Copy link
Collaborator

All code changes are pretty much at physics level and all approved ok at sub components.

LarissaReames-NOAA pushed a commit to LarissaReames-NOAA/ufs-weather-model that referenced this pull request Dec 5, 2024
* UFSWM - Thompson-Eidhammer microphysics code formatting
  * FV3 - Thompson-Eidhammer microphysics code formatting
    * ccpp-physics - Thompson-Eidhammer microphysics code formatting
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. 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.

5 participants