Skip to content

Add dense inverse and its tests#1440

Open
Plusero wants to merge 14 commits into
mainfrom
dense-inversion
Open

Add dense inverse and its tests#1440
Plusero wants to merge 14 commits into
mainfrom
dense-inversion

Conversation

@Plusero

@Plusero Plusero commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Relates to #78

Selective inverse for block sparse matrices is a component for the Uncertainty Quantification of State Estimation. Dense inverse is a function needed by selective inverse for block sparse matrices.

Changes proposed in this PR

  • Extract reusable forward and backward substitution logic from SparseLUSolver.
  • Reuse the extracted substitution logic in DenseLUFactor.
  • Add dense_inverse to DenseLUFactor.
  • Add tests for dense_inverse, including reuse of an existing test matrix.
  • Let dense LU factorization take lvalue ref, for better readability and removal of capturing into void.

Plusero added 8 commits June 12, 2026 16:25
…seLUFactor

Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
@Plusero Plusero added the feature New feature or request label Jun 16, 2026
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
@Plusero

Plusero commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

build-test-release / build-cpp-test-macos (release, macos-26-intel) fails in installing uv. It doesn't seem to originate from my commits. Should I do something on my side, or leave it as is?

@mgovers

mgovers commented Jun 17, 2026

Copy link
Copy Markdown
Member

build-test-release / build-cpp-test-macos (release, macos-26-intel) fails in installing uv. It doesn't seem to originate from my commits. Should I do something on my side, or leave it as is?

looks like a connection issue. rerunning to double-check

@Plusero

Plusero commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Clang-tidy failed. Will fix this afternoon.

Plusero and others added 5 commits June 17, 2026 15:06
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
Signed-off-by: jhe <JialinHe_Plusero@outlook.com>
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants