-
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 complex op #37918
add complex op #37918
Conversation
Thanks for your contribution! |
984b641
to
7b86947
Compare
7b86947
to
fe13986
Compare
fe13986
to
01874a0
Compare
01874a0
to
dbdd25f
Compare
8179e79
to
31a5ef4
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
Complex op has 2 inputs that may require gradient computation. But we have to test with cases when only one of them requires gradient computation. So 'no_grad_set' is used in op test cases. @Shixiaowei02 |
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
PR types
New features
PR changes
OPs
Describe
paddle.complex
. (it supports broadcasting!)Examples