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

Shared physics YSU PBL #1991

Merged
merged 11 commits into from
Feb 1, 2024
Merged

Conversation

weiwangncar
Copy link
Collaborator

@weiwangncar weiwangncar commented Jan 24, 2024

TYPE: enhancement, new feature

KEYWORDS: shared physics, ysu pbl

SOURCE: internal

DESCRIPTION OF CHANGES:
Add YSU PBL scheme from the shared physics directory, which is ccpp-compliant.

Solution:
The CCPP-compliant YSU PBL code will be residing in phys/physics_mmm/ directory. Currently it is a copy of the code from the future MMM-physics repo.

LIST OF MODIFIED FILES:
M Registry/Registry.EM_COMMON
M arch/postamble
M arch/preamble
M clean
M main/depend.common
M phys/Makefile
M phys/module_bl_ysu.F
M phys/module_pbl_driver.F
M phys/module_physics_init.F
A phys/physics_mmm/bl_ysu.F90

TESTS CONDUCTED:

  1. Option runs before and after the change do not produce identical results with 'configure -d'. It does produce bit-for-bit results when running with 36, 12 and 1 processors.
  2. The Jenkins tests have passed.

RELEASE NOTE: Add YSU PBL scheme from the future shared MMM-physics repository.

…needed)

- Add make clean for new directory phys/physics_mmm. Need to remove .F90 so that
  files in phys/physics_mmm/ will be kept.
- Since logical variable cannot be written out as global attribute, remove it.
…and regime, were

  removed from the call to ysu, and flag_qc flag was added.
- modified module_pbl_driver.F to add errmsg and errflg.
- modified phys/Makefile to comile ccpp-compliant code in phys/physics_mmm/
- add dependencies to main/depend.common
@weiwangncar weiwangncar requested review from a team as code owners January 24, 2024 05:47
@weiwangncar
Copy link
Collaborator Author

The regression test results:

Test Type              | Expected  | Received |  Failed
= = = = = = = = = = = = = = = = = = = = = = = =  = = = =
Number of Tests        : 23           24
Number of Builds       : 60           57
Number of Simulations  : 158           150        0
Number of Comparisons  : 95           86        0

Failed Simulations are: 
None
Which comparisons are not bit-for-bit: 
None

dudhia
dudhia previously approved these changes Feb 1, 2024
@smileMchen
Copy link
Collaborator

@weiwangncar
Wei,
Option runs before and after the change do not produce identical results with 'configure -d', --- is this a concern? If the physics scheme itself is not changed, I expect that the results should be the same. What could be the possible reason that the results are not identical? Thanks.

@weiwangncar
Copy link
Collaborator Author

@weiwangncar Wei, Option runs before and after the change do not produce identical results with 'configure -d', --- is this a concern? If the physics scheme itself is not changed, I expect that the results should be the same. What could be the possible reason that the results are not identical? Thanks.

Yes, it is somewhat a concern. One could argue that the newly polished YSU routine may be more robust. I do plan to follow up on this a bit more during the testing phase.

smileMchen
smileMchen previously approved these changes Feb 1, 2024
@weiwangncar weiwangncar dismissed stale reviews from smileMchen and dudhia via 66e6a56 February 1, 2024 21:25
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.

This list of modified files is now reduced.

@weiwangncar weiwangncar merged commit e86f1a3 into wrf-model:develop Feb 1, 2024
1 of 2 checks passed
@Plantain
Copy link

Plantain commented Apr 3, 2024

Does a clean build actually succeed after these changes? After this patchset we receive:
module_bl_ysu.f90(4): error #7002: Error in opening the compiled module file. Check INCLUDE paths. [BL_YSU]
use bl_ysu

It looks like it also has problems on Jenkins, but I can't see the build output.

@weiwangncar
Copy link
Collaborator Author

@Plantain Our regression tests have worked. Is your system case-sensitive? Can your computer tell .F90 from .f90 files?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants