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

Remove usage of deprecated function Argument.ufl_element() #441

Merged
merged 7 commits into from
Mar 23, 2022

Conversation

jorgensd
Copy link
Sponsor Member

@garth-wells
Copy link
Member

Does 'Compatible with: FEniCS/ufl#80' mean that FEniCS/ufl#80 is required for this PR?

@jorgensd
Copy link
Sponsor Member Author

Does 'Compatible with: FEniCS/ufl#80' mean that FEniCS/ufl#80 is required for this PR?

No, but it is needed if FEniCS/ufl#80 gets merged. It does not seem likely at the moments, as there are different opinions on the subject of ufl.Arguments returning the underlying ufl_element, or if one should access it through the function space.

@garth-wells garth-wells merged commit 4ef4b31 into main Mar 23, 2022
@garth-wells garth-wells deleted the mscroggs/tidy branch March 23, 2022 21:40
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.

2 participants