Skip to content

Conversation

celdred
Copy link
Contributor

@celdred celdred commented Jun 21, 2019

This adds proper L2 pullbacks (ie integral-preserving, designed for n-forms), by defining new elements at the UFL level and introducing an "L2 Piola" mapping. These elements then share the same FIAT/FiNAT classes as the original elements, since from the perspective of the tabulator the pullback doesn't matter.

Requires firedrakeproject/tsfc#194 and FEniCS/ufl#4.

@celdred celdred requested review from wence-, dham and thomasgibson June 26, 2019 19:25
This was referenced Jun 26, 2019
Copy link
Member

@dham dham left a comment

Choose a reason for hiding this comment

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

This looks fine to me. Obviously this wants to wait for the other two PRs to land.

dham
dham previously approved these changes Jun 27, 2019
@wence-
Copy link
Contributor

wence- commented Jul 12, 2019

I've opened the UFL PR: https://bitbucket.org/fenics-project/ufl/pull-requests/114

@miklos1
Copy link
Member

miklos1 commented Aug 26, 2019

The two PRs this depends on are already merged.

@wence-
Copy link
Contributor

wence- commented Sep 6, 2019

I just squashed everything, let's see that it runs against current master.

Copy link
Contributor

@wence- wence- left a comment

Choose a reason for hiding this comment

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

I think this is fine.

@dham dham merged commit ee4bbbe into master Sep 9, 2019
@dham dham deleted the l2pullbacks branch September 9, 2019 08:38
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.

4 participants