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

fix NSSL MP init issue when initialized from other microphysics schemes #2098

Merged
merged 17 commits into from
Jan 31, 2024

Conversation

JiliDong-NOAA
Copy link
Contributor

@JiliDong-NOAA JiliDong-NOAA commented Jan 18, 2024

Commit Queue Requirements:

  • Fill out all sections of this template.
  • All sub component pull requests have been reviewed by their code managers.
  • Run the full RT suite (compared to current baselines) on either Hera/Derecho/Hercules AND have committed the log to my PR branch.
  • Add list of all failed regression tests in "Regression Tests" section.

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

  • Critical Bugfix (This PR contains a critical bug fix and should be prioritized.)
  • High (This PR contains a feature or fix needed for a time-sensitive project (eg, retrospectives, implementations))
  • Normal

Blocking Dependencies

Git Issues Fixed By This PR

#2100

Changes

Subcomponent (with links)

Input data

  • No changes are expected to input data.
  • Changes are expected to input data:
    • New input data.
    • Updated input data.

Regression Tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:
FAILED REGRESSION TESTS

Libraries

  • Not Needed
  • Needed
    • Create separate issue in JCSDA/spack-stack asking for update to library. Include library name, library version.
    • Add issue link from JCSDA/spack-stack following this item

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • Jet
    • Gaea
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
    • Completed
  • opnReqTest
    • N/A
    • Log attached to comment

@JiliDong-NOAA
Copy link
Contributor Author

RegressionTests_hera.log

@jkbk2004
Copy link
Collaborator

@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.

@JiliDong-NOAA
Copy link
Contributor Author

@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.

@BrianCurtis-NOAA
Copy link
Collaborator

@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.

@zach1221
Copy link
Collaborator

@JiliDong-NOAA Please sync up your branch. We'd like to begin testing against this PR.

@JiliDong-NOAA
Copy link
Contributor Author

@JiliDong-NOAA Please sync up your branch. We'd like to begin testing against this PR.

Done.

@zach1221 zach1221 added the Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. label Jan 30, 2024
@zach1221
Copy link
Collaborator

@BrianCurtis-NOAA I think we're ready to work this PR. I'm going to start my tests.

@zach1221
Copy link
Collaborator

Hercules is being a bit slow today. One test left pending, but it may be a while longer.

@zach1221
Copy link
Collaborator

Testing is complete. Let's begin with the merging process on ccpp-physics #157

@JiliDong-NOAA
Copy link
Contributor Author

.gitmodules reverted and FV3 updated

@zach1221
Copy link
Collaborator

@JiliDong-NOAA fv3atm PR is merged. Please update the submodule hash and revert the change in .gitmodule url.
hash: NOAA-EMC/fv3atm@ef04de2

@JiliDong-NOAA
Copy link
Contributor Author

@JiliDong-NOAA fv3atm PR is merged. Please update the submodule hash and revert the change in .gitmodule url. hash: NOAA-EMC/fv3atm@ef04de2

Done.

@zach1221 zach1221 merged commit 10974d9 into ufs-community:develop Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Baseline Change No Baseline Change Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants