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

Coupled stochy fixes for precipitation #416

Merged
merged 2 commits into from
Mar 27, 2020

Conversation

pjpegion
Copy link
Collaborator

The pull request references #415, there was an inconsistency between the partition of the rain and snow and the rain/snow tendency that was later perturbed as part of SPPT.
The update to GFS_MP_generic.F90 makes these consistent.

Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Straightforward changes, made consistently in fv3atm and ccpp-physics. I am approving it, but we merging will require coordination with the ufs-weather-model commit queue. I assume this will change the results for the fv3_stochy/fv3_ccpp_stochy tests?

@pjpegion
Copy link
Collaborator Author

@climbfuji This does not change the current regression test since it only is an issue with the coupled model, and the regression tests are not running anything coupled yet.

@climbfuji
Copy link
Collaborator

@climbfuji This does not change the current regression test since it only is an issue with the coupled model, and the regression tests are not running anything coupled yet.

Of course - I should have known.

@climbfuji
Copy link
Collaborator

Update 03/24/2020: this PR has been pulled into #417 and will be merged automatically as part of it. No need to close this PR, it will be flagged as merged automatically.

@climbfuji climbfuji merged commit 53c1071 into NCAR:master Mar 27, 2020
hannahcbarnes pushed a commit to hannahcbarnes/ccpp-physics that referenced this pull request Aug 3, 2022
…effective radii computation from Thompson MP" and "Fix/improve logic for convective transportable tracers, add GFS_checktracer debugging routine, correct spelling "janic" --> "janjic", fix inconsistent types related with noahmp and log functions" (NCAR#416)

* Add missing active attribute for one variable in ccpp/data/GFS_typedefs.meta
* Correct spelling of janic in CCPP standard names, must be janjic
* Remove invalid optional and intent attributes from DDT metadata in ccpp/data/GFS_typedefs.meta
* Remove support for optional arguments from CCPP framework/metadata
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.

2 participants