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

Fix -check all runtime errors (for CCPP) using zero-size allocations #767

Conversation

grantfirl
Copy link
Collaborator

@grantfirl grantfirl commented Jan 18, 2024

Description

This PR at least partially addresses ufs-community/ufs-weather-model#2023 (at least any runtime errors associated with the CCPP). The strategy is to simply make sure that every variable is allocated, although for variables that had been conditionally-allocated to save memory, they are now being allocated with zero size. This has a ripple effect in other repos where one needs to change from checking whether a variable is allocated to whether the size is > 0.

Note: There was also some cleanup done to remove variables that are declared in GFS_ and CCPP_typedefs.F90 but are not being used anywhere anymore. In addition, the ACTIVE attribute in the metadata files was double-checked for every conditionally-allocated variable.

Issue(s) addressed

ufs-community/ufs-weather-model#2023

Testing

This was tested with the full RT suite on Hera with the -check all compilation flag added in the ufs-weather-model Intel.cmake file. All tests completed successfully with no BL changes.

Dependencies

@grantfirl grantfirl changed the title Fix -check all using zero-size allocations Fix -check all runtime errors (for CCPP) using zero-size allocations Jan 18, 2024
@grantfirl
Copy link
Collaborator Author

This is replaced by #789 that is targeted at the RRFS release branch. This will likely NOT need to go to develop since a longer-term fix is forthcoming.

@grantfirl grantfirl closed this Feb 27, 2024
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.

1 participant