Skip to content

Feature/pseudosqrt tests - #2846

Open
losterbr wants to merge 3 commits into
lballabio:masterfrom
losterbr:feature/pseudosqrt-tests
Open

losterbr wants to merge 3 commits into
lballabio:masterfrom
losterbr:feature/pseudosqrt-tests

Conversation

@losterbr

Copy link
Copy Markdown
Contributor

Summary

Improve pseudoSqrt and rankReducedSqrt test coverage and guard empty inputs.

Changes

  • Add coverage for all pseudoSqrt salvage algorithms:
    • None
    • Spectral
    • Hypersphere
    • LowerDiagonal
    • Higham
    • Principal
  • Add coverage for rank reduction, validation, singular matrices, zero matrices, and oversized rank limits.
  • Verify PSD reconstruction, diagonal preservation, finite outputs, and Principal tolerance behavior.
  • Reject empty matrices explicitly in both public APIs.
  • Correct the normalization diagnostic from “pseudo rows” to “pseudo columns”.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 78.421% (+0.2%) from 78.251% — losterbr:feature/pseudosqrt-tests into lballabio:master

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