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.

Sign up for GitHub

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

Jump to bottom

Remove optional keyword from CCPP metadata (8 hours) #407

Closed
gold2718 opened this issue Oct 18, 2021 · 0 comments · Fixed by #408, NOAA-EMC/fv3atm#416 or ufs-community/ufs-weather-model#892
Closed

Remove optional keyword from CCPP metadata (8 hours) #407

gold2718 opened this issue Oct 18, 2021 · 0 comments · Fixed by #408, NOAA-EMC/fv3atm#416 or ufs-community/ufs-weather-model#892
Labels
capgen bugs, requests, etc. that involve ccpp_capgen ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild enhancement

Comments

@gold2718
Copy link
Collaborator

gold2718 commented Oct 18, 2021 •
edited by climbfuji
Loading

Description

The CCPP Framework committee has decided that the CCPP will not support optional scheme arguments. They have replaced with the active metadata keyword.

Solution

Remove the optional keyword from capgen, prebuild, and all existant metadata files.
@gold2718 gold2718 added enhancement capgen bugs, requests, etc. that involve ccpp_capgen ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild labels Oct 18, 2021
@gold2718 gold2718 mentioned this issue Oct 18, 2021
Closed
@gold2718 gold2718 changed the title Remove optional keyword from CCPP metadata Remove optional keyword from CCPP metadata (8 hours) Oct 18, 2021
This was referenced Oct 25, 2021
Merged
Closed
Closed
This was referenced Nov 2, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
capgen bugs, requests, etc. that involve ccpp_capgen ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove optional arguments from CCPP (capgen metadata parser and ccpp_prebuild scripts) climbfuji/ccpp-framework
Wrapper PR for "Remove optional arguments from CCPP metadata, remove effective radii computation from Thompson MP" and "Fix/improve logic for convective transportable tracers, add GFS_checktracer debugging routine, correct spelling "janic" --> "janjic", fix inconsistent types related with noahmp and log functions" climbfuji/fv3atm
Wrapper PR for "Remove optional arguments from CCPP metadata, remove effective radii computation from Thompson MP" and "Fix/improve logic for convective transportable tracers, add GFS_checktracer debugging routine, correct spelling "janic" --> "janjic", fix inconsistent types related with noahmp and log functions", "Modify opnReqTest related scripts for fv3_gsd test" climbfuji/ufs-weather-model
Remove optional arguments from CCPP framework/metadata climbfuji/fv3atm
Remove optional arguments from CCPP framework/metadata climbfuji/ufs-weather-model
1 participant
@gold2718