search

NCAR / ccpp-framework

Common Community Physics Package (CCPP)
http://www.dtcenter.org/community-code/common-community-physics-package-ccpp/
Other
26 stars 64 forks source link

Update main from feature/capgen 11/09/2021 (stricter checking of units in CCPP metadata), update CODEOWNERS - requires Python 3.6 or newer #415

Closed climbfuji closed 2 years ago

climbfuji commented 2 years ago

Description

Update main from feature/capgen as of 11/09/2021. The feature/capgen branch contains major refactoring of the capgen.py scripts, including the metadata used by the current ccpp_prebuild.py in the UFS.

Except for a stricter checking of units in the CCPP metadata, these updates have no effect on the UFS or the parameterizations in the NCAR ccpp-physics main repository.

Note. The framework updates contain formatted string statements that require Python 3.6 or newer.

User interface changes?: No

Fixes https://github.com/NCAR/ccpp-framework/issues/416

Last-minute change w/o effect on the code whatsoever: add @mkavulich to CODEOWNERS

Associated PRs

https://github.com/NCAR/ccpp-framework/pull/415 https://github.com/NCAR/ccpp-physics/pull/775 https://github.com/NOAA-EMC/fv3atm/pull/422 https://github.com/ufs-community/ufs-weather-model/pull/907

Testing

See https://github.com/ufs-community/ufs-weather-model/pull/907

climbfuji commented 2 years ago

@grantfirl for some reason I can't add you as reviewer to this PR. Maybe you didn't exist or were caught in a space-time bubble while transferring from NCAR to NOAA.

climbfuji commented 2 years ago

Regression testing complete, will merge now.