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

Modified 2 ugwp metadata files to fix ufs-community/ufs-weather-model… #425

Merged
merged 5 commits into from
Dec 1, 2021

Conversation

mdtoy
Copy link
Contributor

@mdtoy mdtoy commented Nov 17, 2021

… 'fhzero reproducibility' issue #908

Description

This PR updates the .gitmodules pointer to NCAR/ccpp-physics branch that fixes a bug in the UGWP drag suite (both "unified_ugwp" and "ugwpv1_gsldrag" schemes) where the changing the "fhzero" (diagnostic bucket) variable in the input.nml would change the forecast, which should not happen. This does change the expected answer from Regression Testing.

Issue(s) addressed

This PR fixes ufs-community/ufs-weather-model issue #908

Testing

The update passed the Operation Requirement Testing. On Hera, see the following log file for the test result:
/scratch1/BMC/wrfruc/mtoy/git_local/ufs-weather-model/tests/OpnReqTests_hera.intel.log

Dependencies

None

@junwang-noaa
Copy link
Collaborator

@mdtoy would you please sync your ccpp physics and fv3atm branches with the latest develop branch?

@climbfuji
Copy link
Collaborator

ccpp-physics hash is correct, ready to merge

@junwang-noaa junwang-noaa merged commit 7e2724d into NOAA-EMC:develop Dec 1, 2021
SamuelTrahanNOAA pushed a commit to SamuelTrahanNOAA/fv3atm that referenced this pull request Jun 13, 2022
…ux init bug fix in CCPP (NOAA-EMC#425)

* change the namelist setting for noahmp
* Consolidate namelist templates for NoahMP tests
Co-authored-by: Helin Wei <Helin.Wei@noaa.gov>
Co-authored-by: Brian Curtis <brian.curtis@noaa.gov>
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.

5 participants