-
Notifications
You must be signed in to change notification settings - Fork 0
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
QLGYRO clean-up #419
Comments
The change is in master? |
Yes, there were multiple changes today, so please test the version that is currently in master. |
We debugged Igor's OMP optimizations for CGYRO and merged them into master. The issue is that Igor (who never makes mistakes but also never builds QLGYRO) is not keeping track of the impact of CGYRO changes on QLGYRO. The same is true of my recent changes. It would be good to move QLGYRO toward being insulated from CGYRO optimizations and refactoring. |
QLGYRO now segfaults running it on corg:
|
Can you please send me the test case and how to run it? |
Or point me to where it is on corg? |
|
Thank you. It looks like the issue happened a while ago -- the last version that works is 06/14, so I will need to track down when it occurred. |
I fixed the seg fault error (the mapper for TGLF had not been updated to account for the change in TGLF a few months ago to have more shaping coefficients; I also updated the CGYRO mapper since only had up to sin3/cos3). Anyways, back to the original test. The new change for the clean-up is only triggered when "cgyro_profile_model_in=2". @jmcclena Can you please run a case to test this and let us know? |
@jmcclena @jcandy In QLGYRO, there is a clean-up routine that is run when pre-processing runs with an input.gacode. The subroutine that is called is "qlgyro_cgyro_deallocate_arrays". This appears to be an exact repeat of "cgyro_clean" with the exception of the line "call parallel_lib_clean". I don't think that it is incorrect to clean the parallel_lib at this time though.
So I have replaced all of the code in "qlgyro_cgyro_deallocate_arrays" with "call cgyro_clean".
Joey, can you please test that this does not break any functionality and let us know? [I can only see that it compiles ok.]
We need to make this work bc the repeat of code from cgyro is breaking every time we make a change in cgyro.
The text was updated successfully, but these errors were encountered: