Skip to content

Small fixes in README.namelist regarding MYNN scheme #2220

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

Open
wants to merge 1 commit into
base: release-v4.7.1
Choose a base branch
from

Conversation

MHBalsmeier
Copy link

The purpose of this pull request is to fix the default of bl_mynn_mixlength, which was wrongly given as 2 in the namelist; and clarify the definition of bl_mynn_cloudmix.

TYPE: bugfix

KEYWORDS: README, namelist, default, MYNN

SOURCE: Max Balsmeier

DESCRIPTION OF CHANGES:
Problem:
The namelist was wrong and/or confusing regarding some options of MYNN.

Solution:
The README was fixed/clarified.

ISSUE: n.a.

LIST OF MODIFIED FILES: run/README.namelist

TESTS CONDUCTED:

  1. n.a.

RELEASE NOTE: Fix and clarification in README.namelist regarding MYNN PBL scheme.

@MHBalsmeier MHBalsmeier requested a review from a team as a code owner May 4, 2025 23:14
@islas islas changed the base branch from develop to release-v4.7.1 May 5, 2025 21:12
Copy link
Collaborator

@dudhia dudhia left a comment

Choose a reason for hiding this comment

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

I meant the bl_mynn_closure section which is in the middle of the PBL options. It should be moved with the other bl_mynn options. Sorry this was for another PR. Please ignore.

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.

2 participants