Skip to content

WIP: ES volume integral#468

Closed
sloede wants to merge 45 commits intomainfrom
es-vol-int
Closed

WIP: ES volume integral#468
sloede wants to merge 45 commits intomainfrom
es-vol-int

Conversation

@sloede
Copy link
Copy Markdown
Member

@sloede sloede commented Mar 14, 2021

TODO

gregorgassner and others added 4 commits March 13, 2021 22:48
…l vol int. dirty first implementation: fluxdifferencing volume integral is hijacked for the first implementation. at the moment only 2d
sloede and others added 4 commits March 14, 2021 09:02
Comment on lines +25 to +29
# TODO: This should be moved somewhere else but that would require more involved
# changes since we wouldn't know `nelements(dg, cache)`...
du_ec = Array{uEltype, 3}(undef, nvariables(equations), nnodes(dg), nelements(dg, cache))
du_cen = similar(du_ec)
cache = (;cache..., du_ec, du_cen)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would (somewhat) violate the calling convention, but could we just pass nelements(dg, cache) or cache to the corresponding create_cache method? Just a thought, I don't think this requires fixing right now, since we might switch to element-wise kernels anyways.

@sloede
Copy link
Copy Markdown
Member Author

sloede commented Mar 14, 2021

@ranocha Thanks for the update... I have to say, though, I'd very much like to find a better name for the new struct once we have figured out whether we want this in Trixi permanently or not...

@sloede
Copy link
Copy Markdown
Member Author

sloede commented Mar 14, 2021

As a second note, I'd like to say that I am not yet sure we should implement such a flexible comparison machinery unless we have more than one application for it... Otherwise, I'd prefer to just implement a special case and then extend it to the current form once we actually do have more than one (or even more than two) different versions to compare...

@ranocha
Copy link
Copy Markdown
Member

ranocha commented Mar 14, 2021

I have to say, though, I'd very much like to find a better name for the new struct once we have figured out whether we want this in Trixi permanently or not...

That's why I added # TODO: This needs a better name... 😅

ranocha and others added 21 commits March 15, 2021 18:23
Co-authored-by: Michael Schlottke-Lakemper <michael@sloede.com>
Co-authored-by: Michael Schlottke-Lakemper <michael@sloede.com>
flux plus dissipation and global Lax-Friedrichs dissipation
These can be used to construct upwind SBP operators. Using the standard central DGSEM volume integral and these fluxes at the interfaces results in upwind SBP operators, which are provably linearly stable for Burgers
@DanielDoehring
Copy link
Copy Markdown
Member

This is superseded by the PRs #2472 #2754 #2764

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants