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

Changed UGWP diagnostic variable declaration intents from 'out' to 'inout' #627

Merged
merged 5 commits into from
Feb 22, 2023

Conversation

mdtoyNOAA
Copy link
Contributor

Description

This pull request fixes Issue #37 of the ufs-community/ccpp-physics repository. The UGWP diagnostic variables (i.e., dusfc_ms, dvsfc_ms, tdaux2d_ms, datauy2d_ms, etc.) are declared with "intent(out)" and initialized in module unified_ugwp.F90, then they are passed to subroutine "drag_suite_run" (in drag_suite.F90). In this subroutine, the UGWP diagnostic variables should have been declared with "intent(inout)" instead of "intent(out)". This PR fixes this.
This PR does not change any results.

Issue(s) addressed

Testing

How were these changes tested? Regression testing and ORT testing.
What compilers / HPCs was it tested with? Intel/Hera
Are the changes covered by regression tests? Yes. (But there are no baseline changes.)
Have the ufs-weather-model regression test been run? On what platform? Yes. Hera/Intel.

  • Will the code updates change regression test baseline? No,
  • Please commit the regression test log files in your ufs-weather-model branch

Dependencies

None.

@grantfirl
Copy link
Collaborator

@mdtoyNOAA ufs-community/ccpp-physics#40 has been merged. Please revert the changes to .gitmodules and update the ccpp/physics submodule pointer in anticipation of merge.

@jkbk2004
Copy link
Collaborator

@mdtoyNOAA ccpp pr was merged. can you update submodule pointer and revert change in gitmodules ?

@mdtoyNOAA
Copy link
Contributor Author

mdtoyNOAA commented Feb 22, 2023 via email

@jkbk2004
Copy link
Collaborator

Yes, but I won’t be able to get to it for about an hour and a half.

On Wednesday, February 22, 2023, JONG KIM @.> wrote: @mdtoyNOAA https://github.com/mdtoyNOAA ccpp pr was merged. can you update submodule pointer and revert change in gitmodules ? — Reply to this email directly, view it on GitHub <#627 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARRVLIA23COD5ZAALPKAPTTWYYTRTANCNFSM6AAAAAAU2554VU . You are receiving this because you were mentioned.Message ID: @.>
-- ————————— Michael Toy, Research Scientist CIRES, University of Colorado Boulder NOAA Global Systems Laboratory 325 Broadway, Boulder, Colorado @.*** +1- <(303)%20497-4061>505-609-8782

Sure, no problem!

@mdtoyNOAA
Copy link
Contributor Author

Done. I hope I did it correctly.

@jkbk2004 jkbk2004 merged commit d9e619e into NOAA-EMC:develop Feb 22, 2023
@mdtoyNOAA mdtoyNOAA deleted the develop_drag_suite_intent_mods branch March 9, 2023 20:38
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.

Possible correction to declaration intent for diagnostic variables in drag_suite.F90
5 participants