-
Notifications
You must be signed in to change notification settings - Fork 92
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
Make sigz array in MATXSR allocatable like in GROUPR #274
base: develop
Are you sure you want to change the base?
Conversation
@whaeck I tested on my machine (Linux gfortran-12) without error... Can I get more error output? |
When the CI fails it is often difficult to figure out why. I will give this a try on a machine that has gfortran-9 installed to see if I can replicate the issue. As a side note: since the arrays that you made allocate-able are local in a function you should not have to deallocate them, but you may want to try an input file with two matxsr runs in them to verify it is fine (in the past I have been bitten when trying to reallocate an already allocated array in a previous module call). In this case I think we're fine but it's better to be sure. |
Good point. I added statements to make sure those arrays aren't allocated. |
Interesting: I've cloned your repo, compiled it with gcc 9.3 and am now running the tests. Test 3 fails (still waiting on test 30 to run) but also test 2 is failing (that's a ccccr test). |
Even more interesting, for test 3 I get the following error message with gcc-9.3: At line 1082 of file /users/wim/njoy-testing/NJOY2016/src/matxsr.f90 Error termination. Backtrace: I'll update to your latest change - which seems to have solved the issue - from what I can see int he current CI state. |
So, you were right! Test 3 involves 2 materials and it seems the allocated arrays weren't deallocated even when they were out of scope. |
I saw something like that happen before but I wasn't thinking it would actually be the reason behind the errors :-) Test 2 still fails on my side though. I'll have to look into it some more. SInce it is a CCCCR related thing, it is independent of this PR though. |
I checked test 2 with gfortran-12 and gfortran-9.5, both ran without error.... |
Test 2 failing is unrelated to this PR so I'm not worried about that in this context. I have to check a few more things, but this might be good to go. |
When having more than 15 sigz values processed in GROUPR, MATXSR would simply crash because of hard coded array sigz of size 15.
This was now adapted to the GROUPR method, where those arrays are allocatable.
This PR includes already #270 .