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

ZM clean up round 2 #88

Merged
merged 10 commits into from
Apr 16, 2024
Merged

Conversation

cacraigucar
Copy link
Collaborator

More cleanup identified during CCPP standard name meetings

Includes preliminary metadata for ZM (ignore for now as I know it is not complete - next review should include the entire files)

Closes #83

@cacraigucar cacraigucar requested a review from nusbaume March 13, 2024 22:26
@cacraigucar cacraigucar self-assigned this Mar 13, 2024
Copy link
Collaborator

@nusbaume nusbaume left a comment

Choose a reason for hiding this comment

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

Looks good! Just a few changes requested in the momtran code.

Also I didn't review the metadata files, but if you would like me to at this stage please let me know. Thanks!

zhang_mcfarlane/zm_conv_momtran.F90 Outdated Show resolved Hide resolved
zhang_mcfarlane/zm_conv_momtran.F90 Outdated Show resolved Hide resolved
zhang_mcfarlane/zm_conv_momtran.F90 Outdated Show resolved Hide resolved
@cacraigucar cacraigucar requested a review from nusbaume April 15, 2024 20:40
Copy link
Collaborator

@nusbaume nusbaume left a comment

Choose a reason for hiding this comment

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

I have one last comment change request, but it doesn't require a re-review. Otherwise everything looks great now, thanks!

Comment on lines 133 to 134
real(kind_phys) :: pguall(ncol,pver,num_winds) ! Combined apparent force from updraft on PG U winds
real(kind_phys) :: pgdall(ncol,pver,num_winds) ! Combined apparent force from downdraft on PG U winds
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think on should be after PG here:

Suggested change
real(kind_phys) :: pguall(ncol,pver,num_winds) ! Combined apparent force from updraft on PG U winds
real(kind_phys) :: pgdall(ncol,pver,num_winds) ! Combined apparent force from downdraft on PG U winds
real(kind_phys) :: pguall(ncol,pver,num_winds) ! Combined apparent force from updraft PG on U winds
real(kind_phys) :: pgdall(ncol,pver,num_winds) ! Combined apparent force from downdraft PG on U winds

@cacraigucar cacraigucar merged commit 1642d5c into ESCOMP:main Apr 16, 2024
1 check passed
@cacraigucar cacraigucar deleted the atmos_phys_zm_ccpp branch April 16, 2024 22:17
@cacraigucar cacraigucar restored the atmos_phys_zm_ccpp branch April 16, 2024 22:20
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.

Change directory name from zm to zhang_mcfarlane
2 participants