Skip to content

Comments

GWDO2026 : # 2 of GWDO and SHINHONG PBL PR#2285

Open
Songyou184 wants to merge 3 commits intowrf-model:developfrom
Songyou184:GWDO2026
Open

GWDO2026 : # 2 of GWDO and SHINHONG PBL PR#2285
Songyou184 wants to merge 3 commits intowrf-model:developfrom
Songyou184:GWDO2026

Conversation

@Songyou184
Copy link

@Songyou184 Songyou184 commented Feb 19, 2026

The first line should be a single-line "purpose" for this change

TYPE: choose one of [bug fix, enhancement, new feature, feature removed, no impact, text only]

KEYWORDS: 5 to 10 words related to commit, separated by commas

SOURCE: Either "developer's name (affiliation)" .XOR. "internal" for a WRF Dev committee member

DESCRIPTION OF CHANGES:
Problem:
Generally or specifically, what was wrong and needed to be addressed?

Solution:
What was down algorithmically and in the source code to address the problem?

LIST OF MODIFIED FILES:
m Registry/Registry.EM_COMMON
m arch/Externals.cfg
m dyn_em/module_first_rk_step_part1.F
m phys/module_bl_gwdo.F
m phys/module_pbl_driver.F

TESTS CONDUCTED:

  1. Do mods fix problem? How can that be demonstrated, and was that test conducted?
  2. Are the Jenkins tests all passing?

RELEASE NOTE: Include a stand-alone message suitable for the inclusion in the minor and annual releases. A publication citation is appropriate.

 	modified:   Registry/Registry.EM_COMMON
 	modified:   arch/Externals.cfg
 	modified:   dyn_em/module_first_rk_step_part1.F
 	modified:   phys/module_bl_gwdo.F
 	modified:   phys/module_pbl_driver.F
 	modified:   .gitmodules
 	modified:   arch/Externals.cfg
@Songyou184 Songyou184 requested a review from a team as a code owner February 20, 2026 20:43
@weiwangncar
Copy link
Collaborator

@Songyou184 @dudhia It doesn't seem that the GWDO and ShinHong PBL code are separated. Both PRs 2285 and 2286 contain changes for both options.

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