Skip to content

add missing functions#387

Merged
davidkoski merged 2 commits intomainfrom
linalg-missing
Apr 13, 2026
Merged

add missing functions#387
davidkoski merged 2 commits intomainfrom
linalg-missing

Conversation

@davidkoski
Copy link
Copy Markdown
Collaborator

Proposed changes

Fix #385

  • eig - eigenvalues and eigenvectors
  • eigh - eigenvalues/eigenvectors of symmetric/Hermitian matrix
  • eigvals - eigenvalues only
  • eigvalsh - eigenvalues of symmetric/Hermitian matrix
  • pinv - pseudo-inverse

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING document
  • I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the necessary documentation (if needed)

- eig - eigenvalues and eigenvectors
- eigh - eigenvalues/eigenvectors of symmetric/Hermitian matrix
- eigvals - eigenvalues only
- eigvalsh - eigenvalues of symmetric/Hermitian matrix
- pinv - pseudo-inverse
- logcumsumexp
- segmentedMm (segmented_mm)
- permutation
- fftshift
- ifftshift
@davidkoski davidkoski marked this pull request as ready for review April 6, 2026 16:19
Copy link
Copy Markdown
Member

@angeloskath angeloskath left a comment

Choose a reason for hiding this comment

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

Looks great!

defer { mlx_io_writer_free(writer) }

_ = evalLock.withLock {
_ = try withError {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oops. Did I miss that in the refactoring PR a couple weeks ago 🤔

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

You may have missed it, but I wrote it :-)

@davidkoski davidkoski merged commit 49d73ab into main Apr 13, 2026
8 checks passed
@davidkoski davidkoski deleted the linalg-missing branch April 13, 2026 23:46
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.

[BUG] missing linalg functions

2 participants