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 deprecated fem.FunctionSpace function #2767

Closed
garth-wells opened this issue Sep 11, 2023 · 3 comments · Fixed by #2818
Closed

Remove deprecated fem.FunctionSpace function #2767

garth-wells opened this issue Sep 11, 2023 · 3 comments · Fixed by #2818
Assignees
Labels
task A task to perform
Milestone

Comments

@garth-wells
Copy link
Member

And rename FunctionSpaceBase -> FunctionSpace.

@garth-wells garth-wells added the task A task to perform label Sep 11, 2023
@garth-wells garth-wells added this to the v0.8.0 milestone Sep 11, 2023
@francesco-ballarin
Copy link
Member

Would it be a good idea to do this immediately after FEniCS/ufl#197 is merged?

My rationale is that I will probably have to go through my codes and double check that every single time I create a function space I do not use ufl.legacy elements anymore (and replace them with basix ones, if I were not using them already). If this case change were enforced while I was already doing these checks, in a single pass I would be killing two birds with one stone.

@mscroggs
Copy link
Member

Would it be a good idea to do this immediately after FEniCS/ufl#197 is merged?

This seems like a sensible approach to me

@francesco-ballarin francesco-ballarin self-assigned this Oct 16, 2023
@francesco-ballarin
Copy link
Member

OK, I'll do this after #2784 is merged.

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

Successfully merging a pull request may close this issue.

3 participants