ProjectTorreyPines / FUSE.jl

FUsion Synthesis Engine
https://fuse.help/
Apache License 2.0
18 stars 0 forks source link

TJLF handling of multiple ions #537

Open orso82 opened 6 months ago

orso82 commented 6 months ago

@tomneiser can you please check how we are handling the ion energy flux coming out of TJLF?

To get the total Qi I think we should be doing a sum over all of the ion energy fluxes, but I don't see that...

https://github.com/ProjectTorreyPines/FUSE.jl/blob/00667cfe23d774aedd1ca9f0191fd559c2c22d64/src/actors/transport/tglf_actor.jl#L112

orso82 commented 6 months ago

pinging @TimSlendebroek as well

tomneiser commented 6 months ago

Take a look at this regression test for accurate parsing of results: https://github.com/ProjectTorreyPines/TJLF.jl/blob/master/test/runtests_regressions.jl

orso82 commented 6 months ago

@tomneiser can you please review my interpretation here: https://github.com/ProjectTorreyPines/TJLF.jl/commit/cfebbadedd038ce1e2b77b474d3b282590879737