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

Replace temporary ccpp_const_get_idx subroutine with official CCPP-framework version #215

Open
nusbaume opened this issue Mar 4, 2025 · 0 comments
Labels
cleanup This issue indicates an opportunity for code cleanup. enhancement New feature or request

Comments

@nusbaume
Copy link
Collaborator

nusbaume commented Mar 4, 2025

Now that NCAR/ccpp-framework#622 has been merged, the CCPP-framework itself provides a mechanism to look up a constituent array index based off the standard name(s). This provides an opportunity to replace the temporary ccpp_const_get_idx subroutine in to_be_cccppized/ccpp_const_utils.F90 with an official framework version, which will inherit all of the associated benefits (more easily portable, better tested, etc.).

Note that this will require an update to CAM-SIMA as well, both to update the framework tag to access the new routine, and to replace the use of ccpp_const_get_idx in CAM-SIMA itself.

@nusbaume nusbaume added cleanup This issue indicates an opportunity for code cleanup. enhancement New feature or request labels Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup This issue indicates an opportunity for code cleanup. enhancement New feature or request
Projects
Status: To Do
Development

No branches or pull requests

1 participant