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

Update support/HAFS from NCAR dtc/hwrf-physics 2020/06/19 #5

Conversation

climbfuji
Copy link

@climbfuji climbfuji commented Jun 19, 2020

This PR adds the HWRF physics to fv3atm. All development was made in the NCAR fork, branch dtc/hwrf-physics. Commit log from this branch:

04/06/2020 MONINEDMF https://github.com/NCAR/ufs-weather-model/pull/15 -> https://github.com/NCAR/ufs-weather-model/pull/35
    git cherry-pick e6fe22ffde6425800b7ed2e1cf3748b606806d28
    git cherry-pick a99d5e73f94f5ffb3bd9bbb1a3dc04171026b6d7
    git cherry-pick 5570528f22a9815437e0ef5b413435a8b9fd5881
    git cherry-pick 2bb15be8cc15c69c178b6e443c0ab8c0d75ef318

04/08/2020 RRTMG https://github.com/NCAR/ufs-weather-model/pull/30 -> https://github.com/NCAR/ufs-weather-model/pull/36
    git cherry-pick 5ab446fe423d9a7ba0172ec782e6346c47c98116
    git cherry-pick 1cbc515c6dff00e729523ff6f922e046179d7dcf
    git cherry-pick c6f51f5090411c737a8d79a27bbf08b8d9b00c28
    git cherry-pick 2a43ab170916fbe60c3d918ff16a510928964e51
    git cherry-pick 73f5dcbc93ae297a6a9880988a8f7a7109f99c2e

04/13/2020 SAS https://github.com/ufs-community/ufs-weather-model/pull/94 -> https://github.com/NCAR/ufs-weather-model/pull/39
    (entire PR)

04/16/2020 FA MP updates https://github.com/NCAR/ufs-weather-model/pull/41 (replaced https://github.com/NCAR/ufs-weather-model/pull/25)
    (entire PR)

05/04/2020 FA/HWRF RRTMG/HWRF SAS bugfixes https://github.com/NCAR/ufs-weather-model/pull/45
    (entire PR)

05/20/2020 update from HAFS community https://github.com/NCAR/ufs-weather-model/pull/48
    (entire PR)

06/18/2020 update from HAFS community https://github.com/NCAR/ufs-weather-model/pull/56
    (entire PR)

Associated PRs:
#5
hafs-community/fv3atm#5
hafs-community/ccpp-physics#2

For regression testing information, see below NOT YET.

climbfuji and others added 30 commits April 1, 2020 14:43
…n configuration required other than one namelist value - hurr_pbl = T)
…ist for HWRF PBL, add test to rt_ccpp_dtc.conf
…ccpp_regional_c768_FA_update_moist and fv3_ccpp_regional_c768_HWRF_PBL
…dd-moninedmf

dtc/hwrf-physics: combined version of HAFS/GFS moninedmf scheme (hybrid EDMF PBL)
…est namelist templates and run scripts, use own HWRF regression test baseline
mzhangw
mzhangw previously approved these changes Jun 19, 2020
Copy link

@mzhangw mzhangw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

ligiabernardet
ligiabernardet previously approved these changes Jun 19, 2020
@climbfuji climbfuji dismissed stale reviews from ligiabernardet and mzhangw via dadbb0c June 19, 2020 21:21
@climbfuji
Copy link
Author

Regression testing on hera.intel against official EMC baseline (using rt.conf): all tests passed.

rt_hera_intel.log
RegressionTests_hera.intel.log
Compile_hera.intel.log

@climbfuji
Copy link
Author

Regression testing on hera.intel using rt_hwrf_physics.conf against previous regression test baseline created with the NCAR dtc/hwrf-physics branch: all tests passed.

rt_hwrf_physics_hera_intel.log

@climbfuji
Copy link
Author

@BinLiu-NOAA all three PRs listed as associated PRs above are ready to merge from our side. Please let me know next week when you are ready and we can get started with the merge.

Thanks!

@BinLiu-NOAA
Copy link

Thanks, @climbfuji ! Once @mzhangw pushed her feature/hwrf_physics branch of HAFS into the hafs-community fork, we will start some workflow level regression tests. After that, we can then merge these pull requests.

Thanks!

@climbfuji climbfuji force-pushed the update_hafs_from_dtc_hwrf_physics_20200619 branch from 47cc1b1 to 348af8e Compare June 25, 2020 03:00
@climbfuji
Copy link
Author

@BinLiu-NOAA all regression tests passed on hera with the cleaned up code in these three PRs. Do you want me to commit
RegressionTests_hera.intel.log and Compile_hera.intel.log as part of this PR ?

  • against official EMC baseline using rt.conf

rt_hera_intel.log
RegressionTests_hera.intel.log
Compile_hera.intel.log

  • on hera.intel using rt_hwrf_physics.conf: first create temporary baseline, then verify against it

rt_hera_intel_hwrf_physic_create.log
rt_hera_intel_hwrf_physic_verify.log

@BinLiu-NOAA
Copy link

@climbfuji Please feel free to commit in these log files as you see fit. Thanks!

@climbfuji
Copy link
Author

@climbfuji Please feel free to commit in these log files as you see fit. Thanks!

Done - are we ready to commit?

BinLiu-NOAA
BinLiu-NOAA previously approved these changes Jun 26, 2020
Copy link

@BinLiu-NOAA BinLiu-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready to merge after the fv3atm level PR goes into support/HAFS.

@climbfuji
Copy link
Author

@BinLiu-NOAA ufs-weather-model is ready, please check again. Thanks!

Copy link

@BinLiu-NOAA BinLiu-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Ready to merge. Thanks!

@climbfuji climbfuji merged commit 16f6559 into hafs-community:support/HAFS Jun 26, 2020
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