Skip to content

Default SuperLU_DIST to PARMETIS ordering and expose ParSymbFact - #4665

Merged
jhale merged 5 commits into
mainfrom
jhale/superlu-parmetis-default
Oct 19, 2026
Merged

jhale merged 5 commits into
mainfrom
jhale/superlu-parmetis-default

Leave ParSymbFact off by default

6473661
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Oct 19, 2026 in 58s

Annotations

Check warning on line 664 in cpp/dolfinx/la/superlu_dist.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace alternative operator "and" with "&&".

See more on https://sonarcloud.io/project/issues?id=FEniCS_dolfinx&issues=AaFS7tUlhqlKp0Vud0q1&open=AaFS7tUlhqlKp0Vud0q1&pullRequest=4665

Check warning on line 548 in cpp/dolfinx/la/superlu_dist.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace alternative operator "and" with "&&".

See more on https://sonarcloud.io/project/issues?id=FEniCS_dolfinx&issues=AaFS7tUlhqlKp0Vud0qy&open=AaFS7tUlhqlKp0Vud0qy&pullRequest=4665

Check failure on line 585 in cpp/dolfinx/la/superlu_dist.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 27 to the 25 allowed.

See more on https://sonarcloud.io/project/issues?id=FEniCS_dolfinx&issues=AaFS7tUlhqlKp0Vud0qz&open=AaFS7tUlhqlKp0Vud0qz&pullRequest=4665

Check warning on line 316 in python/test/unit/la/test_superlu_dist.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this exception test to have only one invocation possibly throwing an exception.

See more on https://sonarcloud.io/project/issues?id=FEniCS_dolfinx&issues=AaFS3O1FeUbertIuCswt&open=AaFS3O1FeUbertIuCswt&pullRequest=4665

Check warning on line 629 in cpp/dolfinx/la/superlu_dist.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace alternative operator "or" with "||".

See more on https://sonarcloud.io/project/issues?id=FEniCS_dolfinx&issues=AaFS7tUlhqlKp0Vud0q0&open=AaFS7tUlhqlKp0Vud0q0&pullRequest=4665

Check warning on line 662 in cpp/dolfinx/la/superlu_dist.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace alternative operator "and" with "&&".

See more on https://sonarcloud.io/project/issues?id=FEniCS_dolfinx&issues=AaFS3OodeUbertIuCsws&open=AaFS3OodeUbertIuCsws&pullRequest=4665