Open DanielDoehring opened 1 year ago
To summarize the discussions in https://github.com/trixi-framework/Trixi.jl/pull/1629 I list some open points:
Outer datastructures for non-internal variables in ViscousContainers: https://github.com/trixi-framework/Trixi.jl/blob/474410507181fa93cbe8201da4ade03311f09104/src/solvers/dgsem_tree/container_viscous_2d.jl#L3-L15 Usage of Tuple also for noninternal datastructures?
ViscousContainer
Tuple
Different reinitialize_containers! for parabolic cache (?) to avoid the if in https://github.com/trixi-framework/Trixi.jl/blob/474410507181fa93cbe8201da4ade03311f09104/src/solvers/dgsem_tree/containers.jl#L31-L35
reinitialize_containers!
Alternative to specialization via flux_viscous? https://github.com/trixi-framework/Trixi.jl/blob/dc0dc1d220e79d31465818acae235adf69ecddea/src/solvers/dgsem_tree/dg_2d_parabolic.jl#L518-L523
flux_viscous
Corresponding PR: https://github.com/trixi-framework/Trixi.jl/pull/1754
To summarize the discussions in https://github.com/trixi-framework/Trixi.jl/pull/1629 I list some open points:
Outer datastructures for non-internal variables in
ViscousContainer
s: https://github.com/trixi-framework/Trixi.jl/blob/474410507181fa93cbe8201da4ade03311f09104/src/solvers/dgsem_tree/container_viscous_2d.jl#L3-L15 Usage ofTuple
also for noninternal datastructures?Different
reinitialize_containers!
for parabolic cache (?) to avoid the if in https://github.com/trixi-framework/Trixi.jl/blob/474410507181fa93cbe8201da4ade03311f09104/src/solvers/dgsem_tree/containers.jl#L31-L35Alternative to specialization via
flux_viscous
? https://github.com/trixi-framework/Trixi.jl/blob/dc0dc1d220e79d31465818acae235adf69ecddea/src/solvers/dgsem_tree/dg_2d_parabolic.jl#L518-L523