Skip to content

Conversation

@aayushg55
Copy link
Contributor

  • Created new pinv() operator for matrix pseudo-inverse using SVD in Reduced mode. Optional rcond parameter which is used to define the cutoff for small singular values. Singular values smaller than rcond * largest_singular_value are set to 0.
  • Changed SVDMode::REDUCED interface so that U and VT tensors passed in must be of reduced size instead of the full size. That is, instead of U being of shape ... x m x m, it should have shape ... x m x k where k=min(m,n). Similarly VT should be of shape ... x k x n.
  • Added det() documentation page

Resolves #487

@cliffburdick
Copy link
Collaborator

/build

@aayushg55 aayushg55 requested a review from cliffburdick August 28, 2024 15:34
@cliffburdick
Copy link
Collaborator

/build

Copy link
Collaborator

@cliffburdick cliffburdick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice work

@coveralls
Copy link

Coverage Status

coverage: 93.538% (+0.2%) from 93.386%
when pulling cfea0a3 on pinv
into d9053d6 on main.

@cliffburdick
Copy link
Collaborator

/build

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.

[FEA] Add pseudo-inverse operator

4 participants