-
Notifications
You must be signed in to change notification settings - Fork 419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add native implementation of inv (Gauss-Jordan Elimination) #16255
Conversation
rahulghangas
commented
Aug 19, 2020
•
edited
Loading
edited
- Add inplace implementation
- Testing
@ben-albrecht needs label gsoc:LinearAlgebra |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a few comments.
halt("Wrong rank for matrix inverse"); | ||
|
||
if !isSquare(A) then | ||
halt("Matrix inverse only supports square matrices"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be a compilerError
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
compilerError doesn't work if the argument to inv is a call to another function that returns a 2-D matrix. halt does
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That seems like a bug in the compiler to me... Do you have a short example demonstrating this?
Co-authored-by: Ben Albrecht <ben-albrecht@users.noreply.github.com>
Co-authored-by: Ben Albrecht <ben-albrecht@users.noreply.github.com>
Is this still a todo? If not, and this is ready for a review, could you update the PR message and let us know, so we move forward with this. If it is still a todo let us know if you can/want to work on it soon. |
Based on silence, I am closing this PR. @rahulghangas -- Please feel free to reopen when you have the time to work on this. |