WDM scheme minor bug fixes#1057
Merged
dudhia merged 1 commit intowrf-model:release-v4.1.4from Feb 4, 2020
Merged
Conversation
Collaborator
|
@dudhia Can we see some differences in a model run? |
Collaborator
Author
|
I asked if they have any images.
…On Thu, Jan 23, 2020 at 11:48 AM weiwangncar ***@***.***> wrote:
@dudhia <https://github.com/dudhia> Can we see some differences in a
model run?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1057?email_source=notifications&email_token=AEIZ77FD5AZWWUQOMKWCE3DQ7HRAVA5CNFSM4KKMIIVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJYNMDY#issuecomment-577820175>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEIZ77H3JJD65FV2TUOJ46TQ7HRAVANCNFSM4KKMIIVA>
.
|
Contributor
|
@dudhia |
Contributor
|
@dudhia |
Collaborator
Author
|
Not sure where Sunny works now. Sooya was at KIAPS, not sure now. |
Contributor
|
@weiwangncar @dudhia
|
Collaborator
|
@dudhia I guess to most users, what impact these changes have to surface precipitation would be of some interest. Do we know? Can we get a statement from the contributors? |
weiwangncar
approved these changes
Feb 4, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TYPE: bug fix
KEYWORDS: WDM5, WDM6, WDM7
SOURCE: Kyo-Sun Lim (Kyungpook National University) and Sooya Bae (KIAPS).
DESCRIPTION OF CHANGES:
Error correction based on Lei et al. (JGR, 2020)
-Problem: melting processes of snow/graupel mass and number concentration do not occur
at the same time.
-Correction: melting of snow/graupel changes the snow/graupel/rain mass and rain number
concentration at the same time.
-Effect: Generation of rain number concentrations would decrease.
-Problem: Cloud water number concentration does not add into the CCN number concentration.
-Correction: Adding the cloud water number concentration into the CCN number concentration.
ISSUE: none
LIST OF MODIFIED FILES:
module_mp_wdm5.F
module_mp_wdm6.F
module_mp_wdm7.F
TESTS CONDUCTED:

Passed Jenkins
WDM6 tested versus master. Diffs small for surface fields including rainfall after 12 hrs of June 2001 test case (shown).
RELEASE NOTE: Melting of snow/graupel now changes the snow/graupel/rain mass and rain number concentration at the same time in all WDM schemes, provided by Kyo-Sun Lim (Kyunpook National University, South Korea).