-
Notifications
You must be signed in to change notification settings - Fork 16
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
Core radiation fix #3624
base: develop
Are you sure you want to change the base?
Core radiation fix #3624
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is going to take me a bit of time to go through. Here are a few things to start
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I have no knowledge of where most of this maths come from so could we have a few references or documents for that?
The code itself looks fine although the tests need fixing
Quality Gate passedIssues Measures |
Ok there are tests for calculate_z_species that need changing/updating/removing because you've removed the function here. There are some other failures too because of name changes |
The core radiation was not calculated correctly because when accessing the tables from ADAS stored in PROCESS, Lz and Zeff where accessed with a reference temperature in keV instead of eV.
Besides the radiation calculation fix, a few checks related to the effective charge profiles and the average Z_eff value across the plasma were added.
Also added a scaling factor to use non-uniform electron temperature and electron density along the closed flux lines in the plasma core.