Closed patrickersing closed 3 months ago
This checklist is meant to assist creators of PRs (to let them know what reviewers will typically look for) and reviewers (to guide them in a structured review process). Items do not need to be checked explicitly for a PR to be eligible for merging.
NEWS.md
.Created with :heart: by the Trixi.jl community.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.31%. Comparing base (
2dfde7f
) to head (0111b0a
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I also tested convergence by running the test elixirs with refined meshes and comparing the gauge point data (point_data
) from the callback to the analytical solution (exact_data
).
Results for tree_1d_dgsem/elixir_euler_source_terms.jl : |
initial_refinement_level |
norm(point_data - exact_data) |
---|---|---|
4 | 1.587e-1 | |
6 | 3.883e-10 | |
8 | 3.966e-13 |
Results for tree_3d_dgsem/elixir_euler_source_terms.jl : |
initial_refinement_level |
norm(point_data - exact_data) |
---|---|---|
2 | 2.701e-1 | |
3 | 9.439e-2 | |
4 | 2.170e-5 |
This PR extends the
TimeSeriesCallback
onTreeMesh
to 1D and 3D. The existing routines worked pretty much out of the box and the extension only required one additional function. Since many of the methods intime_series_dg_2d.jl
are actually independent of mesh and dimension, I would suggest to relocate them intotime_series_dg.jl
. Furthermore, since most functions forTreeMesh
are independent of the dimension, I think it makes sense to split these off into a filetime_series_dg_tree.jl
to avoid code duplication.