-
Notifications
You must be signed in to change notification settings - Fork 250
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
fix NSSL MP init issue when initialized from other microphysics schemes #2098
Conversation
@JiliDong-NOAA can you sync up branches? Once branches are synced up, I will try to follow up to combine this PR to #2104 at physics PR level. |
Thanks. All submodules synced up. |
@JiliDong-NOAA EPIC would like to get to this PR next. Once a merge of the current UFSWM PR completes, please make sure that you updates hashes from FV3 once that is updated. |
@JiliDong-NOAA Please sync up your branch. We'd like to begin testing against this PR. |
Done. |
@BrianCurtis-NOAA I think we're ready to work this PR. I'm going to start my tests. |
Hercules is being a bit slow today. One test left pending, but it may be a while longer. |
Testing is complete. Let's begin with the merging process on ccpp-physics #157 |
.gitmodules reverted and FV3 updated |
@JiliDong-NOAA fv3atm PR is merged. Please update the submodule hash and revert the change in .gitmodule url. |
Done. |
Commit Queue Requirements:
PR Information
Description
This is the fix from Ted Mansell @MicroTed on NSSL MP initialization to check if there is missing number of concentration for any non-zero hydrometers mixing ratio. NSSL MP will calculate N from mixing ratio in the first timestep if missing value is found. This fix will guarantee that NSSL MP enters the correct warm/cold start initializations.
Commit Message
fix NSSL MP init issue when initialized from other microphysics schemes
Priority
Blocking Dependencies
Git Issues Fixed By This PR
#2100
Changes
Subcomponent (with links)
Input data
Regression Tests:
FAILED REGRESSION TESTS
Libraries
Testing Log: