Closed andrewwinters5000 closed 4 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.
Attention: Patch coverage is 97.97297%
with 3 lines
in your changes are missing coverage. Please review.
Project coverage is 96.29%. Comparing base (
e205b06
) to head (40f6547
).
Files | Patch % | Lines |
---|---|---|
src/solvers/fdsbp_unstructured/fdsbp_2d.jl | 92.11% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thanks a lot! I will merge this and release a new Trixi.jl version when CI passes (whereever it can...)
Great, thanks! It is nice to have finally figured out all this curvilinear upwind stuff. I will close my old PR and delete that (very stale) branch.
This PR replaces #1295 . Implementation of the upwind FDSBP solver on unstructured curvilinear meshes. The solver converges for all available splittings and is free-stream preserving provided the boundary polynomial order from the mesh file, i.e.,
mesh_polydeg
is half the boundary closure order of the SBP operator.