Skip to content

Add support for MYNN PBL scheme in convection-permitting physics suite#503

Merged
kuanchihwang merged 10 commits into
ESCOMP:developmentfrom
kuanchihwang:staging/mynn-pbl-scheme
Jul 1, 2026
Merged

Add support for MYNN PBL scheme in convection-permitting physics suite#503
kuanchihwang merged 10 commits into
ESCOMP:developmentfrom
kuanchihwang:staging/mynn-pbl-scheme

Conversation

@kuanchihwang

@kuanchihwang kuanchihwang commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Tag name

None (Bit for bit)

Originator(s)

kuanchihwang

Description (include the issue title, and the keyword ['closes', 'fixes', 'resolves'] followed by the issue number)

This PR adds the required host model variables for MYNN PBL scheme to the registry. Additional host model control variables (e.g., {initial,restart,branch}_run) are exposed as read-only through CCPP framework.

Describe any changes made to the build system

None

Describe any changes made to the namelist

None

List any changes to the defaults for the input datasets (e.g., initial, boundary datasets)

None

List all files eliminated and why

None

List all files added and what they do

None

List all existing files that have been modified, and describe the changes

M       src/control/cam_control_mod.F90
M       src/control/cam_control_mod.meta
  * Expose model control variables through CCPP framework
M       src/data/registry.xml
  * Add variables required by MYNN PBL scheme to registry

Regression tests

  • derecho/aux_sima_gnu
SMS_Ln9.ne3pg3_ne3pg3_mg37.FADIAB.derecho_gnu.cam-outfrq_se_cslam (Overall: FAIL) details:
  FAIL SMS_Ln9.ne3pg3_ne3pg3_mg37.FADIAB.derecho_gnu.cam-outfrq_se_cslam RUN time=10

Known failing test.

  • derecho/aux_sima_intel
SMS_Ln9.ne3pg3_ne3pg3_mg37.FKESSLER.derecho_intel.cam-outfrq_se_cslam_multitape (Overall: NLFAIL) details:
  FAIL SMS_Ln9.ne3pg3_ne3pg3_mg37.FKESSLER.derecho_intel.cam-outfrq_se_cslam_multitape NLCOMP

Known failing test.

  • derecho/aux_sima_nvhpc

Pass.

Certain physics schemes (e.g., MYNN PBL scheme) really want to know about it.
@kuanchihwang
kuanchihwang temporarily deployed to CI-tests-on-CIRRUS May 18, 2026 05:58 — with GitHub Actions Inactive
@kuanchihwang
kuanchihwang marked this pull request as ready for review May 18, 2026 21:12
@kuanchihwang
kuanchihwang requested a review from nusbaume May 18, 2026 21:13
@kuanchihwang
kuanchihwang temporarily deployed to CI-tests-on-CIRRUS June 4, 2026 19:37 — with GitHub Actions Inactive
@kuanchihwang
kuanchihwang temporarily deployed to CI-tests-on-CIRRUS June 5, 2026 19:21 — with GitHub Actions Inactive
@kuanchihwang
kuanchihwang temporarily deployed to CI-tests-on-CIRRUS June 5, 2026 20:23 — with GitHub Actions Inactive

@nusbaume nusbaume left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @kuanchihwang!

@nusbaume
nusbaume requested a review from jimmielin June 14, 2026 03:05
@kuanchihwang
kuanchihwang temporarily deployed to CI-tests-on-CIRRUS June 18, 2026 18:39 — with GitHub Actions Inactive
Comment thread src/data/registry.xml Outdated
Comment thread src/data/registry.xml Outdated
Note that this is a deviation from the ESM Standard Names.
@kuanchihwang
kuanchihwang temporarily deployed to CI-tests-on-CIRRUS June 29, 2026 21:58 — with GitHub Actions Inactive

@jimmielin jimmielin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @kuanchihwang!

I saw you used lowercase o3 in atmospheric_physics. I think that should be fine, I just chatted with @peverwhee who confirmed that standard names should be case-insensitive, but I would just test it to make sure it is working as intended.

@kuanchihwang
kuanchihwang merged commit 0d0d630 into ESCOMP:development Jul 1, 2026
17 checks passed
@kuanchihwang
kuanchihwang deleted the staging/mynn-pbl-scheme branch July 1, 2026 06:31
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