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 #85

Closed
wants to merge 5 commits into from

Conversation

pjpegion
Copy link
Collaborator

The pull request references ufs-community/ufs-s2s-model#58, 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_physics_driver.F90 makes these consistent.

@climbfuji
Copy link
Collaborator

I approved the corresponding ccpp-physics change. Merging will have to be coordinated with the ufs-weather-model commit queue.

@climbfuji
Copy link
Collaborator

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

@climbfuji
Copy link
Collaborator

This PR was merged as part of #86 - for some reason, it doesn't show up as "merged" automatically. Please double-check, but I think this PR can be closed.

@pjpegion
Copy link
Collaborator Author

pjpegion commented Mar 27, 2020 via email

@climbfuji
Copy link
Collaborator

Hi Dom, I see the updates are now in develop, the PR can be closed Thanks, Phil

Thanks! Importantly, the commits still show you as author, so you get the credit for doing the work (and not me for pulling them in).

@junwang-noaa
Copy link
Collaborator

PR #86 was committed, close this PR.

climbfuji pushed a commit to climbfuji/fv3atm that referenced this pull request Apr 26, 2021
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.

3 participants