-
Notifications
You must be signed in to change notification settings - Fork 660
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
Shared physics YSU PBL #1991
Conversation
…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
The regression test results:
|
@weiwangncar |
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. |
There was a problem hiding this 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.
Does a clean build actually succeed after these changes? After this patchset we receive: It looks like it also has problems on Jenkins, but I can't see the build output. |
@Plantain Our regression tests have worked. Is your system case-sensitive? Can your computer tell .F90 from .f90 files? |
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:
RELEASE NOTE: Add YSU PBL scheme from the future shared MMM-physics repository.