-
Notifications
You must be signed in to change notification settings - Fork 2
Add fast density calculation to IMAS.sources #275
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
base: master
Are you sure you want to change the base?
Conversation
Add fast density calculation to IMAS.sources Add max_fast_frac parameter to prevent thermal density becoming negative.
Undo "return refreeze" (not sure how that happened.
diffusivities add calculation of to transport.jl
Minor bug fix
minor bug fixed for fast ion quasineutrality
minor bug fix
@orso82 I'm happy with the pull request now, and I think it's ready for this to be merged. |
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.
looks mostly good. Some styling/typing suggestions and a deepcopy that I am not sure is necessary
q_density_difference = ne .- sum(ion.density .* avgZ(ion) for ion in cp1d.ion if ion !== ion0) | ||
if hasdata(ion0, :density_fast) | ||
q_density_difference .-= .-ion0.density_fast .* avgZ(ion0) | ||
q_density_difference .-= ion0.density_fast .* avgZ(ion0) |
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.
good find!
Co-authored-by: Orso Meneghini <[email protected]>
Co-authored-by: Orso Meneghini <[email protected]>
Co-authored-by: Orso Meneghini <[email protected]>
@orso82 Can you verify that the refreezes in |
Remove refreezes from fast_particle_profiles
Add fast density calculation to IMAS.sources
Add max_fast_frac parameter to prevent thermal density becoming negative.