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

ccpp_prebuild: (re-)introduce optional attribute #540

Closed
climbfuji opened this issue Mar 1, 2024 · 0 comments · Fixed by #552
Closed

ccpp_prebuild: (re-)introduce optional attribute #540

climbfuji opened this issue Mar 1, 2024 · 0 comments · Fixed by #552
Assignees
Labels
capgen-unification ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild enhancement

Comments

@climbfuji
Copy link
Collaborator

Description

PR #529 adds the optional attribute to parameterization metadata for capgen, to be used in conjunction with the active attribute in the host model metadata. Additional logic is introduced in the auto-generated caps and rules are imposed on parameterizations to pass potentially unallocated data from the host model to the physics.

Solution

For the transition to capgen, we should reintroduce the optional attribute in ccpp_prebuild (it had it before, but it was used differently). This will require updates to the physics schemes and to the CCPP technical documentation.

Alternatives (optional)

Do nothing and bake the optional-attribute change into the switch to capgen. This is a bad idea, since we want to minimize the changes that need to be made when switching from ccpp_prebuild to capgen.

Related to (optional)

#526

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
capgen-unification ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild enhancement
Projects
capgen unification
  
main (Done)
Development

Successfully merging a pull request may close this issue.

1 participant