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

Update UFL element interface: move elements to FInAT #302

Merged
merged 16 commits into from
Nov 15, 2023

Conversation

mscroggs
Copy link
Contributor

@mscroggs mscroggs commented Oct 16, 2023

This is an extension of #300.

In FInAT/FInAT#111, I moved the elements from ufl.legacy to finat.ufl. This PR updates tsfc to use the elements from there.

This is all a consequence of FEniCS/ufl#197.

This PR is coupled with firedrakeproject/firedrake#3166, FInAT/FInAT#111, and NGSolve/ngsPETSc#7.

@mscroggs mscroggs marked this pull request as draft October 16, 2023 16:23
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.

fix the imports and this is good to go.

requirements-git.txt Outdated Show resolved Hide resolved
requirements-git.txt Outdated Show resolved Hide resolved
connorjward
connorjward previously approved these changes Nov 15, 2023
Copy link
Contributor

@connorjward connorjward left a comment

Choose a reason for hiding this comment

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

This is good.

Copy link
Member

@JDBetteridge JDBetteridge left a comment

Choose a reason for hiding this comment

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

Don't worry about adding these I'll add the suggestions to a batch once I've checked the other branches 🙂

requirements-git.txt Outdated Show resolved Hide resolved
requirements-git.txt Outdated Show resolved Hide resolved
@JDBetteridge JDBetteridge merged commit b4b8ca5 into master Nov 15, 2023
0 of 4 checks passed
@JDBetteridge JDBetteridge deleted the mscroggs/newfl-legacy2 branch November 15, 2023 13:34
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