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

Chsp physics update, global latlon grid, post changes and CCN-IN followup bugfix #104

Merged
merged 20 commits into from
Apr 28, 2020

Conversation

junwang-noaa
Copy link
Collaborator

@junwang-noaa junwang-noaa commented Apr 21, 2020

In the commit:

  1. CHSP from GFSv16 package is added to gfsphysics and ccpp/physics
  2. add capability to output global_latlon to netcdf file
  3. land surface model information is added to the history file, post lib is updated to 8.0.6
  4. Add CCN-IN followup bugfix in GFS_typedefs.F90
  5. Bugfix for Intel 18 on wcoss_cray, remove Intel 16 options for gaea
  6. CMEPS integration changes

Related PRs:
ccpp/physics #440
#104
NEMS #55
ufs-weather-model #108

@@ -4275,7 +4309,7 @@ subroutine GFS_physics_driver &
Statein%pgr, Statein%phil, clw, Stateout%gq0, &
Stateout%gt0, Stateout%gu0, Stateout%gv0, rain1, &
kbot, ktop, kcnv, islmsk, Statein%vvl, ncld, &
Diag%hpbl, hflx, evap, ud_mf, dt_mf, cnvw, cnvc, &
Diag%hpbl, hflxq, evapq, ud_mf, dt_mf, cnvw, cnvc,&
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am sure you have a reason for it, but I don't understand why this change is only for shalcnv, but not for samfshalcnv (or any other convection scheme)?

Copy link
Contributor

Choose a reason for hiding this comment

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

It is because only shalcnv among convection schemes needs kinematic sensible heat flux (hflx) and latent heat flux (evap) for the cloud base mass flux. The other convection schemes do not use hflx & evap.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks, got it! We need to make sure - I'll create an issue to check - that none of the additional convection schemes in CCPP that are not in IPD use hflx & evap.

Copy link
Collaborator

Choose a reason for hiding this comment

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

junwang-noaa and others added 2 commits April 24, 2020 20:20
Co-authored-by: Denise Worthen <Denise.Worthen@noaa.gov>
@@ -8,6 +8,7 @@ module sso_coorde
use machine, only: kind_phys
real(kind=kind_phys),parameter :: pgwd = 1._kind_phys
real(kind=kind_phys),parameter :: pgwd4 = 1._kind_phys
logical debugprint = .false.
Copy link
Collaborator

Choose a reason for hiding this comment

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

@junwang-noaa
Copy link
Collaborator Author

Final RT passed, code is ready for commit after review. Thanks

Copy link
Contributor

@JongilHan66 JongilHan66 left a comment

Choose a reason for hiding this comment

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

Changes for the canopy heat storage parameterization (CHSP), thermal roughness length, and increased background diffusivity and mixing length in the inversion layers look fine.

Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

I did raise one question that remains unanswered (https://github.com/NOAA-EMC/fv3atm/pull/104/files?file-filters%5B%5D=.F90&file-filters%5B%5D=.f&file-filters%5B%5D=.f90&file-filters%5B%5D=.meta&file-filters%5B%5D=dotfile#r414570311), but this shouldn't delay the commit. Otherwise this is a bunch of important changes!

@junwang-noaa
Copy link
Collaborator Author

junwang-noaa commented Apr 28, 2020 via email

Copy link
Collaborator

@DeniseWorthen DeniseWorthen left a comment

Choose a reason for hiding this comment

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

Looks great. All my changes are present as expected.

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.

6 participants