Remove references to physcons module from SCM#653
Remove references to physcons module from SCM#653scrasmussen wants to merge 1 commit intoNCAR:mainfrom
Conversation
68ef7be to
c2e06ac
Compare
hertneky
left a comment
There was a problem hiding this comment.
Code changes look good to me. Only 1 minor comment to take or leave.
scm/src/scm_physical_constants.F90
Outdated
| real(kind=dp),parameter:: con_avgd =6.0221415e23_dp | ||
| real(kind=dp),parameter:: con_amd =28.9644_dp !< molecular wght of dry air (\f$g/mol\f$) | ||
| real(kind=dp),parameter:: con_amw =18.0154_dp | ||
| real(kind=dp),parameter:: con_amo3 =47.9982_dp |
There was a problem hiding this comment.
should we add the description for this one that's being added?
!< molecular wght of o3 (\f$g/mol\f$)
There was a problem hiding this comment.
I'll add that, thanks!
c2e06ac to
3493ce7
Compare
|
Were all new additions to scm_physical_constants.F90/meta from UFS? (e.g. did you have to come up with any new metadata?) |
dustinswales
left a comment
There was a problem hiding this comment.
Need to remove the intent attribute from the host-model metadata.
Other than that this looks fine.
Thanks for making these changes!
scm/src/scm_physical_constants.meta
Outdated
| dimensions = () | ||
| type = real | ||
| kind = kind_phys | ||
| intent = in |
There was a problem hiding this comment.
The intent attribute is not needed for host-model metadata.
There was a problem hiding this comment.
Fixed, thanks for the feedback!
423bce0 to
3f8d05a
Compare
The metadata changes here are all ones that I came up with (based on the comments in the physics and physcon files). I was able to find existing definitions for a lot of physcon variables but not for these references. I may have missed something but I did check the |
SOURCE: Soren Rasmussen, NSF NCAR
DESCRIPTION OF CHANGES:
ISSUE: #652
TESTS CONDUCTED: Compared CI output of unoptimized version of upstream with this PR. All results were identical