-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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 new API/OP: paddle.linalg.triangular_solve #36714
add new API/OP: paddle.linalg.triangular_solve #36714
Conversation
Thanks for your contribution! |
558addd
to
2a783a8
Compare
2a783a8
to
d62922e
Compare
d62922e
to
0930b77
Compare
0930b77
to
62c533c
Compare
62c533c
to
39eb297
Compare
39eb297
to
ef7e4b1
Compare
ef7e4b1
to
2c24183
Compare
7ade8de
to
b312dcb
Compare
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.
LGTM for PR-CI-OP-benchmark.
int M = b->dims()[b_dim_size - 2]; | ||
int N = b->dims()[b_dim_size - 1]; |
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.
Tensor->dims()[] wll return int64_t, use static_cast ?
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.
Done, thx
int M = b->dims()[b_dim_size - 2]; | ||
int N = b->dims()[b_dim_size - 1]; |
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.
Tensor->dims()[] wll return int64_t, use static_cast ?
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.
Done, thx
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.
lgtm
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.
LGTM
# -x3 = 5 | ||
|
||
import paddle | ||
import numpy as np |
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.
numpy没用到。
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.
下个PR顺带删掉~ thx
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.
LG API
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.
LGTM
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.
LGTM
* add new API: paddle.linalg.triangular_solve * add new API/OP: paddle.linalg.triangular_solve * add new API/OP: paddle.linalg.triangular_solve * fix comment
ignore test_triangular_solve.py because PaddlePaddle/Paddle#36714 havn't cherry-pick yet.
* add new API: paddle.linalg.triangular_solve * add new API/OP: paddle.linalg.triangular_solve * add new API/OP: paddle.linalg.triangular_solve * fix comment
PR types
New features
PR changes
APIs
Describe
Computes the solution of a system of equations with a triangular coefficient matrix
x
andmultiple right-hand sides
y
.Input
x
andy
is 2D matrices or batches of 2D matrices. If the inputs are batches, the outputsis also batches.
Args:
Returns:
Examples: