-
Notifications
You must be signed in to change notification settings - Fork 23
Implement dpnp.linalg.lu_solve()
2D inputs
#2575
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
base: master
Are you sure you want to change the base?
Conversation
dpnp.linalg.solve()
2D inputs dpnp.linalg.lu_solve()
2D inputs
Array API standard conformance tests for dpnp=0.19.0dev4=py313h509198e_19 ran successfully. |
View rendered docs @ https://intelpython.github.io/dpnp/pull/2575/index.html |
depends=dep_evs, | ||
) | ||
_manager.add_event_pair(ht_ev, piv_copy_ev) | ||
# MKL lapack uses 1-origin while SciPy uses 0-origin |
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.
I guess SciPy also uses MKL to call getrs
, so it seems unclear for me.
6a3bae1
to
b1aed58
Compare
See Also | ||
-------- | ||
:obj:`dpnp.linalg.lu_factor` : LU factorize a matrix. |
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.
Please add See also
section to lu_factor
docstring also
assert_raises(ValueError, dpnp.linalg.lu_factor, a, check_finite=True) | ||
|
||
|
||
class TestLuSolve: |
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.
Any plans to enable cupy tests?
This PR suggests adding
dpnp.linalg.lu_solve()
for 2D arrays similar to scipy.linalg.lu_solveSupport for ND inputs will be added in the next phase.