-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add support for P4estMesh to subcell-limiting
#121
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. |
I'm thinking about whether it's useful to keep this default implementation of Trixi.jl/src/solvers/dgsem_tree/dg_2d_subcell_limiters.jl Lines 1853 to 1858 in 1b2ce66
What do you think about that @amrueda ? EDIT: On the other hand, it is useful for boundary conditions like |
True, I was also thinking recently about this routine. I think it is better to remove it because it can lead a user to think that the bound computation takes into account their BCs. However, sometimes it makes sense to just use the inner solution. Is it possible to activate this computation with an input argument and keep it inactive as default? |
Yesterday, I revised the implementation of this routine. It is much more flexible and cleaner now. |
- Fixes bug that P4estMesh calls the right `characteristic_boundary_state` routine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Just a minor comment:
Co-authored-by: Andrés Rueda-Ramírez <aruedara@uni-koeln.de>
Adding support for subcell limiting with P4estMesh.
TODOs:
-> Different order in calculating of the boundary fluxes yields slightly different (machine precision) results