-
Notifications
You must be signed in to change notification settings - Fork 261
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
Feature/gfdlmpv3 #2503
base: develop
Are you sure you want to change the base?
Feature/gfdlmpv3 #2503
Conversation
@dustinswales can you sync up branch? we can start working on this pr. also, we are considering to combine in #2621 to this pr. |
…r-model into feature/gfdlmpv3
Done. Do you want me to pull in https://github.com/JessicaMeixner-NOAA/ufs-weather-model/tree/WW3_nochangesync? |
@dustinswales Yes, please go ahead to pull in pull in https://github.com/JessicaMeixner-NOAA/ufs-weather-model/tree/WW3_nochangesync |
Done. |
@dustinswales some update is needed at physics level: /glade/derecho/scratch/jongkim/FV3_RT/rt_23671/compile_atm_debug_dyn32_intel/err
|
@jkbk2004 Sorry, a bug that snuck in during the review process, which was after I ran the full RTs. |
@dustinswales I see some more failures:
While this gets updated, we will let #2487 and #2625 move first. |
@jkbk2004 I reran the full set of RTs and everything passes except for these two cases: |
@dustinswales You error message ( |
@DeniseWorthen Thanks. |
The current UWM is using Line 1019 in 6cb9e1d
but for some reason, in your run log I see
|
I see, I think the PR that got pulled in needs the updated WW3 directory... (wasn't sure what 'og pr' was) |
@DeniseWorthen OG is shorthand for the way too verbose ORG. |
@dustinswales I think Jessica's original PR (2621) should have no baseline changes though, when using the new input. |
@dustinswales We can let #2621 go first maybe by Wednesday. I will double check input file and test on both hera and hercules. |
I have returned from leave, please let me know if anything is needed from me for this PR. |
Commit Queue Requirements:
Description:
This PR contains host side changes to accommodate a new cloud microphysics parameterization in the physics.
For the physics refactoring, all of the configuration/namelist info is moved into a common module that is referenced by both MP versions and a shared fv_sat_adj.F90. There are default values for the scheme configurations defined within this module, and overwritten by the namelist, if provided. Both the v1 and v3 rely on some of the same parameters, but with different default values. This is handled by introducing a pre-processor directive into the new configuration module.
Commit Message:
Priority:
Git Tracking
Sub component Pull Requests:
UFSWM Blocking Dependencies:
Changes
Input data Changes:
Library Changes/Upgrades:
Testing Log: