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 #229

Merged
merged 24 commits into from
Nov 25, 2024

Conversation

grantfirl
Copy link
Collaborator

@grantfirl grantfirl commented Oct 29, 2024

This work is the same as #179, but targeting ufs/dev.

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

This PR includes:

  1. The addition of parameterized kind: REAL -> real(kind_phys)
  2. Consistent indentation
  3. Removal of GOTO statements
  4. Physical constants are now coming from the host.

This is expected to change results for any RTs that use Thompson MP because of point 4.

AndersJensen-NOAA and others added 21 commits December 7, 2023 14:01
…nd it will be consistent with convective precipitation"
…entrations.F90 due to passing in PI as variable
[production/RRFS.v1] physics updates for RRFS.v1 code freeze
…_alloc_RRFS

[production/RRFS.v1] Fix 'check all' with 0-size allocations
@grantfirl grantfirl requested a review from climbfuji November 15, 2024 16:04
Copy link

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Please fix the trailing whitespace, other than that good to go.

physics/MP/Thompson/mp_thompson.F90 Outdated Show resolved Hide resolved
@jkbk2004
Copy link

test are all done at ufs-community/ufs-weather-model#2482. @grantfirl @dustinswales @rhaesung can you merge this pr?

@rhaesung rhaesung merged commit 6e0467a into ufs-community:ufs/dev Nov 25, 2024
3 checks passed
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.

7 participants