-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix allocations of P4estMesh2D BCs #1636
Conversation
Review checklistThis 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. Purpose and scope
Code quality
Documentation
Testing
Performance
Verification
Created with ❤️ by the Trixi.jl community. |
Maybe it makes sense to adapt the 3D case Trixi.jl/src/solvers/dgsem_p4est/dg_3d.jl Lines 357 to 359 in 27d4fd1
also correspondingly (also allocations are not yet observed) function calc_boundary_flux!(cache, t, boundary_condition::BC, boundary_indexing,
mesh::P4estMesh{3},
equations, surface_integral, dg::DG) where {BC} Similarly, one could also add this to Trixi.jl/src/solvers/dgmulti/dg_parabolic.jl Lines 211 to 212 in 27d4fd1
Trixi.jl/src/solvers/dgmulti/dg.jl Lines 463 to 464 in 27d4fd1
Trixi.jl/src/solvers/dgsem_unstructured/dg_2d.jl Lines 316 to 318 in 27d4fd1
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1636 +/- ##
==========================================
- Coverage 92.94% 90.80% -2.14%
==========================================
Files 416 416
Lines 34065 34100 +35
==========================================
- Hits 31661 30963 -698
- Misses 2404 3137 +733
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Thanks for your quick review! I checked the places you mentioned before submitting this PR. They do not need any treatment since they don't trigger Julia's heuristic to avoid specialization. |
Closes #1635