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

Framework should provide a debugging feature to compare array dimensions in ccpp_prebuild.py #405

Closed
climbfuji opened this issue Oct 4, 2021 · 0 comments · Fixed by #404 or ufs-community/ufs-weather-model#850
Closed

Framework should provide a debugging feature to compare array dimensions in ccpp_prebuild.py #405

climbfuji opened this issue Oct 4, 2021 · 0 comments · Fixed by #404 or ufs-community/ufs-weather-model#850
Labels
ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild enhancement

Comments

@climbfuji
Copy link
Collaborator

climbfuji commented Oct 4, 2021 •
edited
Loading

Description

The framework should be able to compare array dimensions in the auto-generated caps between the actual size of the arrays and the size inferred from the dimensions in the metadata.

Solution

These comparisons will be slowing down the code and therefore should only be activated when an appropriate --debug flag (or similar) is provided on the command line. Arrays that are not "active" (i.e. whose active attribute evaluates to .false.) must be skipped.

This solution will be implemented first in ccpp_prebuild.py and later in capgen.py (see #406).
@climbfuji climbfuji added enhancement capgen bugs, requests, etc. that involve ccpp_capgen capgen-unification ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild post-capgen-unification and removed capgen bugs, requests, etc. that involve ccpp_capgen capgen-unification post-capgen-unification labels Oct 4, 2021
@climbfuji climbfuji changed the title Framework should provide a debugging feature to compare array dimensions Framework should provide a debugging feature to compare array dimensions in ccpp_prebuild.py Oct 4, 2021
This was referenced Oct 4, 2021
Closed
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
ccpp_prebuild bugs, requests, etc. that involve ccpp_prebuild enhancement
Projects
None yet
Milestone
No milestone
1 participant
@climbfuji