Skip to content

dtc/develop: update from EMC 2019/11/27#360

Merged
climbfuji merged 8 commits into
NCAR:dtc/developfrom
climbfuji:dtc_develop_udpate_from_emc_20191127
Dec 3, 2019
Merged

dtc/develop: update from EMC 2019/11/27#360
climbfuji merged 8 commits into
NCAR:dtc/developfrom
climbfuji:dtc_develop_udpate_from_emc_20191127

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented Nov 27, 2019

This PR contains:

  • updates from EMC, i.e. from ccpp-physics master (gfdlmp, satmedmfvdifq, fractional landmask)
  • bugfix for GSD drag suite: add missing initialization of height variable, remove unused variables
  • Thompson MP: improve diagnostic messages for negative qv/qr/qs/... and tendency limiter, bugfix for calculating number concentrations (if first moment = mass is zero, set second moment = number concentration to zero)

Associated PRs see below, for regression testing see NCAR/ufs-weather-model#9.

@climbfuji
Copy link
Copy Markdown
Collaborator Author

…d tendency limiter, bugfix for calculating number concentrations
@climbfuji climbfuji marked this pull request as ready for review December 2, 2019 18:34
Copy link
Copy Markdown

@mdtoy mdtoy left a comment

Choose a reason for hiding this comment

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

I approve changes to drag_suite.F90.

Comment thread physics/drag_suite.F90
enddo
enddo
!
! calculate mid-layer height (zl), interface height (zq), and layer depth (dz2).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not a big deal, but this comment no longer reflects what the code is doing.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I actually left it on purpose (and just commented out the parts that are no longer calculated, zq and dz2) in case Mike wants to put it back later.

Copy link
Copy Markdown
Collaborator

@grantfirl grantfirl left a comment

Choose a reason for hiding this comment

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

Approved

@climbfuji climbfuji merged commit f895fc0 into NCAR:dtc/develop Dec 3, 2019
@climbfuji climbfuji deleted the dtc_develop_udpate_from_emc_20191127 branch June 27, 2022 03:12
hannahcbarnes pushed a commit to hannahcbarnes/ccpp-physics that referenced this pull request Aug 3, 2022
Add metadata for three constants that are required for the changes in PR NCAR#525.
The motivation behind these changes is that physical parameterizations should receive constants (e.g. gravitational acceleration) from the host model via the argument list instead of importing them from some Fortran module or defining them locally. This ensures consistency and enhances interoperability.
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