Closed bennibolm closed 1 month 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
with its PR number.Created with :heart: by the Trixi.jl community.
Attention: Patch coverage is 73.75000%
with 84 lines
in your changes are missing coverage. Please review.
Project coverage is 75.26%. Comparing base (
9b64eab
) to head (98c2144
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Due to issues with the code coverage test, continue this PR with a branch on upstream Trixi and not on my fork. https://github.com/trixi-framework/Trixi.jl/pull/1946
This PR adds subcell limiting support for
Structuredmesh
.Getting the inverse Jacobian makes this PR a bit longer. That's because for TreeMesh,
inverse_jacobian
is fixed within one element, while it is different for each node for StructuredMesh. To avoid something likeand also avoid unnecessary calls of inverse jacobian in TreeMesh simulations, I dispatched for the mesh type and extracted the code
[...]
to an inner function (This is also done for e.g. the routineapply_jacobian!
).After this PR, it will be straight forward to add support for P4estMesh.
For now, there is no support for non-conservative systems.
TODO: