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

Maybe this? #2

Conversation

dustinswales
Copy link

No description provided.

@climbfuji
Copy link
Owner

@dustinswales I cherry-picked the commit and resolved the conflict for testing. As I expected, it doesn't work for module variables declared in the group:

[100%] Built target test_host
At line 216 of file /Users/heinzell/work/ccpp-framework/ccpp-framework-debug-20231012/test/capgen_test/test_host.F90
Fortran runtime warning: An array temporary was created
 Available suites are:
1) ddt_suite = test_suites(2)
2) temp_suite = test_suites(1)
 Checking suite temp_suite
 Checking suite ddt_suite
At line 223 of file /Users/heinzell/work/ccpp-framework/ccpp-framework-debug-20231012/test/ct_build/ccpp/ccpp_temp_suite_cap.F90
Fortran runtime error: Index '6' of dimension 1 of array 'temp_level' above upper bound of 5

I'll try to find out if I can add some minimal logic to determine if it's a group module variable and if so, adjust the bounds. Otherwise I'll revert.

@climbfuji
Copy link
Owner

Unfortunately I couldn't find a good solution that was less manual than what I have now.

@climbfuji climbfuji closed this Nov 22, 2023
@dustinswales
Copy link
Author

@climbfuji I just reran the capgen_test using Intel w/o error.
During init:
Screenshot 2023-11-27 at 9 40 46 AM
During run:
Screenshot 2023-11-27 at 9 41 00 AM

@climbfuji
Copy link
Owner

climbfuji commented Nov 27, 2023 via email

climbfuji pushed a commit that referenced this pull request Oct 9, 2024
Unit test actions
Adds capgen unit tests to the list of actions.
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.

2 participants