Skip to content
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

Flexible Wedge Product for 0-forms #44

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Flexible Wedge Product for 0-forms #44

wants to merge 3 commits into from

Conversation

bosonbaas
Copy link
Member

This PR adds an option for different wedge product weighting schemes.

This implementation is based on Griebel et. al 2017 (Upwind Schemes for Scalar Advection-Dominated Problems in the Discrete Exterior Calculus). Through this, we can reproduce several techniques currently used in finite element/finite volume/finite difference methods, like central differences and upwinding.

@jpfairbanks
Copy link
Member

@lukem12345 has this been subsumed by other PRs?

@lukem12345
Copy link
Member

No, in the mean time we have produced wedge products between e.g. primal and dual forms. But methods that perform upwinding have not been implemented.

@jpfairbanks jpfairbanks added the good first issue Good for newcomers label May 14, 2024
@jpfairbanks
Copy link
Member

Now that we are doing more fluid stuff, upwinding seems really helpful. I've labeled this good first issue to get over the finish line.

@jpfairbanks
Copy link
Member

Now that we have a general approach to directional derivatives (using Lie(x,y) as our version of d/dx y), could we use that to do upwinding? We specify a vector field x to be the direction in which we define as "upwind" and then reweight operators like wedge or lie derivatives by scalar multiplication by weights?

@lukem12345
Copy link
Member

Certainly the 1 dimensional case will be easy to implement and test as a preliminary PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants