Skip to content

Remove overly restrictive testing of doxygen comments when looking for scheme tables from scripts/metadata_parser.py#366

Merged
climbfuji merged 4 commits into
NCAR:mainfrom
climbfuji:move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent
May 27, 2021
Merged

Remove overly restrictive testing of doxygen comments when looking for scheme tables from scripts/metadata_parser.py#366
climbfuji merged 4 commits into
NCAR:mainfrom
climbfuji:move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented Apr 15, 2021

This PR removes an overly restrictive and unnecessary test of doxygen comments when looking for scheme tables from scripts/metadata_parser.py.

Associated PRs:

NCAR/ccpp-physics#627
#366
NOAA-EMC/ufsatm#285
ufs-community/ufs-weather-model#529

For regression testing, see ufs-community/ufs-weather-model#529

…r scheme tables from scripts/metadata_parser.py
…move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent
@climbfuji climbfuji changed the title WORK IN PROGRESS - Remove overly restrictive testing of doxygen comments when looking for scheme tables from scripts/metadata_parser.py Remove overly restrictive testing of doxygen comments when looking for scheme tables from scripts/metadata_parser.py May 13, 2021
@climbfuji climbfuji marked this pull request as ready for review May 13, 2021 14:42
Copy link
Copy Markdown
Contributor

@llpcarson llpcarson 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 added 2 commits May 18, 2021 13:05
…ve_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent
…xtent and vice versa disabled, this is now an error
@climbfuji climbfuji merged commit 54b158f into NCAR:main May 27, 2021
@climbfuji climbfuji deleted the move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent branch June 27, 2022 03:05
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.

4 participants