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

Updates to MYNN-EDMF, part II - goes with PR for ccpp-physics #121

Merged
merged 5 commits into from
Feb 15, 2022
Merged

Updates to MYNN-EDMF, part II - goes with PR for ccpp-physics #121

merged 5 commits into from
Feb 15, 2022

Conversation

joeolson42
Copy link
Collaborator

@joeolson42 joeolson42 commented Feb 2, 2022

This PR supports the MYNN-EDMF PR (part I):

  • removes namelist options no longer used (bl_mynn_part and grav_settling)
  • adds new namelist option (bl_mynn_closure) to regulate the closure level of the MYNN - default 2.6
  • fixes a bug which did not allow switching to different mixing length options
  • Adds arrays for TKE budget output when bl_mynn_tkebudget = .true.

Testing

How were these changes tested? In short retros & case studies in debug mode
What compilers / HPCs was it tested with? ifort
Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
Have the ufs-weather-model regression test been run? On what platform?

  • Will the code updates change regression test baseline? If yes, why? Please show the baseline directory below.
  • Please commit the regression test log files in your ufs-weather-model branch

Dependencies

This needs to go with the other MYNN PR:
NOAA-GSL/ccpp-physics#121

@SamuelTrahanNOAA SamuelTrahanNOAA merged commit 1679dc0 into NOAA-GSL:gsl/develop Feb 15, 2022
@joeolson42
Copy link
Collaborator Author

joeolson42 commented Feb 15, 2022 via email

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.

3 participants