-
Notifications
You must be signed in to change notification settings - Fork 38
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
Revamp using new PETSc_jll binaries #99
Conversation
68a826d
to
eaffc9f
Compare
@stevengj would you have any objection to me merging this? I realise it's not complete (in terms of functionality vs the old code), but it would make it easier for us to do more incremental development. |
@JaredCrean2 as well |
@simonbyrne This has been a great start! I tried to see how far we can push the specification of solver options. Here is an example using CG plus PETSc's native AMG preconditioner: 2026516 I think there is some issue with nesting options prefixes - or perhaps I am missing something? |
That's fine with me. Are you planning to manually translate the Petsc headers rather than using Clang.jl? |
I think so: the bigger challenge is keeping track of object lifetimes and figuring out correct return objects, which generated wrappers can't help with. |
Yes, that seems reasonable to me. My initial plan was to do manual header translation, but the Petsc guys got upset with that suggestion and felt that we couldn't possibly keep up with their advanced API changes (see JuliaLang/julia#2645). To the extent that we are focused on providing high-level APIs, however, I don't think that is a problem. |
No description provided.