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

DEC Operators for Barycentric Subdivision #31

Closed
wants to merge 1 commit into from
Closed

Conversation

bosonbaas
Copy link
Member

These are some edits to the hodge star and flat operator in order to account for dual meshes which are generated with barycentric subdivision.

This is a WIP and will need significant cleanup, documentation, and tests. Currently this should reduce to the original operators for circumcentric subdivision.

@bosonbaas
Copy link
Member Author

One more thing that will need to be fixed is that currently the flat operator accepts values at the primal vertices instead of surfaces. This was just due to convenience for getting this working with a constant flow-field, so that will need to be reverted.

@epatters
Copy link
Member

epatters commented Jun 11, 2021

There are lots of possibilities for setting up a flat operator (Hirani suggests 8!). The one Hirani most discusses is a dual-primal flat (sending dual vector fields to a primal 1-forms), so that's the style I implemented, but if it's also convenient to have a primal-primal flat, we could have that too. No need to restrict to only one kind of flat if, e.g., this kind is more convenient for defining constant flow fields.

@epatters epatters changed the title Operators for Barycentric Subdivision DEC Operators for Barycentric Subdivision Jun 15, 2021
@bosonbaas
Copy link
Member Author

This PR is old and was never completed. It would probably be better to start over again on these barycentric subdivision operators, so an issue has been filed #52

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

Successfully merging this pull request may close these issues.

3 participants