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

Make PETSc an optional dependency #2472

Closed
2 tasks done
garth-wells opened this issue Nov 28, 2022 · 6 comments · Fixed by #3123 or #3182
Closed
2 tasks done

Make PETSc an optional dependency #2472

garth-wells opened this issue Nov 28, 2022 · 6 comments · Fixed by #3123 or #3182
Assignees
Labels
high-priority proposal Suggested change or addition suggestion
Milestone

Comments

@garth-wells
Copy link
Member

garth-wells commented Nov 28, 2022

If PETSc was an optional dependency we should, in principle, be able to build under Windows and in particular via conda.

We have native distributed vectors and a beta-version native parallel CSR matrix. We do not have linear solvers but can use SciPy solvers (no MPI).

  • Make PETSc optional on C++ interface
  • Make PETSc optional in Python interface
@garth-wells garth-wells added proposal Suggested change or addition suggestion labels Nov 28, 2022
@chrisrichardson
Copy link
Contributor

I've also used pyamg which is good.

@bhaveshshrimali
Copy link

This would be great! I have also used pypardiso which is quite good.

@garth-wells garth-wells added this to the 0.9.0 milestone Apr 12, 2024
@garth-wells
Copy link
Member Author

We should bump this up in priority - some in industry have license concerns that this change would overcome.

@chrisrichardson chrisrichardson self-assigned this Apr 12, 2024
@chrisrichardson
Copy link
Contributor

https://github.com/FEniCS/dolfinx/tree/chris/petsc-optional is part way there. It is quite easy for C++ but will require more work for Python.
Maybe we should proceed with the C++ part first?

@jorgensd
Copy link
Sponsor Member

Not quite done, as pyproject.toml depends on PETSc4py? @garth-wells

@garth-wells
Copy link
Member Author

Not quite done, as pyproject.toml depends on PETSc4py? @garth-wells

Some details to work through, but can build locally without PETSc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high-priority proposal Suggested change or addition suggestion
Projects
None yet
4 participants