Skip to content

MPI support for PETSc wrapper#950

Merged
ChrisRackauckas merged 6 commits into
SciML:mainfrom
Iddingsite:MPIsupport
Apr 10, 2026
Merged

MPI support for PETSc wrapper#950
ChrisRackauckas merged 6 commits into
SciML:mainfrom
Iddingsite:MPIsupport

Conversation

@Iddingsite

Copy link
Copy Markdown
Contributor

Got some motivation, so this is a follow-up to #949 that adds MPI support for the PETSc wrapper. Should works for all solvers and preconditionners.

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

I did something a bit different than Hypre and used PartitionedArrays.jl. The PR may need further testing, I did not try it on anything serious yet, just basic tests. Needs #949 to be merged before.

Done with Claude's help.

@Iddingsite

Copy link
Copy Markdown
Contributor Author

Lots of false negatives in the tests, this should be ready for review

@ChrisRackauckas

Copy link
Copy Markdown
Member

Interesting... something must've knocked a machine down. Retrigger.

@Iddingsite

Copy link
Copy Markdown
Contributor Author

CI is still NOT happy 😅

Get stuff like this:
Error: The file '/home/chrisrackauckas/github-runners/demeter4-6/_diag/pages/2f0c42b3-9e85-4c10-a7e5-b114a8fc1d8e_f2f8222b-0e16-5a7f-baa4-2e72a0961b8a_1.log' already exists.

@ChrisRackauckas

Copy link
Copy Markdown
Member

It looks like it's all demeter4... I need to go look what happened to that machine.

@ChrisRackauckas ChrisRackauckas merged commit 702f486 into SciML:main Apr 10, 2026
272 of 394 checks passed
@Iddingsite Iddingsite deleted the MPIsupport branch April 10, 2026 20:07
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