Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Eager, Performance optimization] support divide( / operator) to sink to Cpp layer #46329

Merged
merged 42 commits into from
Sep 27, 2022
Merged

[Eager, Performance optimization] support divide( / operator) to sink to Cpp layer #46329

merged 42 commits into from
Sep 27, 2022

Conversation

veyron95
Copy link
Contributor

@veyron95 veyron95 commented Sep 20, 2022 •
edited
Loading

PR types

Performance optimization

PR changes

APIs

Describe

Performance optimization for math operator div, rdiv, truediv, rturediv__

@veyron95
[Eager] math op sink to Cpp level
0519d83
@veyron95
fix ci errors
c81e2ca
@veyron95
draft version
b71919c
@veyron95
draft version
1fbd072
@veyron95
draft version
de98e32
@veyron95
support + and - operator under cpp directly
22fd517
@veyron95
add static test
97204a2
@veyron95
polish code
318c260
@veyron95
promote types or unify right type to left
7110d1c
@veyron95
recover static test case
72303fd
@veyron95
polish code and fix some ci errors
6570443
@veyron95
support complex and polish code
77b5821
@veyron95
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
97dbdc4 
… eager_math_op_path_under_cpp
@veyron95
fix conflicts
b500cdc
@veyron95
fix windows ci errors
ef7a395
@veyron95
fix windows-inference-ci errors
aec4159
@veyron95
polish and fix tests
f07ab50
@veyron95
fix test case
f5f9f55
@veyron95
polish code
c0f2d2f
@veyron95
[Eager, Performance optimization] support multiply( * operator) to si…
a4a184c 
…nk to Cpp layer
@veyron95
rm useless glog
577cdf9
@veyron95
[Eager, Performance optimization] support divide( / and // operator) …
77889be 
…to sink to Cpp layer
@paddle-bot
Copy link

paddle-bot bot commented Sep 20, 2022

你的PR提交成功，感谢你对开源项目的贡献!
请关注后续CI自动化测试结果，详情请参考Paddle-CI手册。
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@veyron95 veyron95 changed the title [Eager, Performance optimization] support divide( / and // operator) to sink to Cpp layer [Eager, Performance optimization] support divide( / operator) to sink to Cpp layer Sep 21, 2022
@veyron95
Merge commit 'refs/pull/45811/head' of https://github.com/PaddlePaddl…
9458cd7 
…e/Paddle into eager_math_op_mul
@veyron95
polish code
7f0c435
@veyron95
fix CI
341f880
@veyron95
polish code
90c64d0
@veyron95
Merge commit 'refs/pull/45811/head' of https://github.com/PaddlePaddl…
7a8fd8b 
…e/Paddle into eager_math_op_mul
@veyron95
polish code
34ffdd1
@veyron95
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
8d03231 
… eager_math_op_mul
@veyron95
Merge commit 'refs/pull/46326/head' of https://github.com/PaddlePaddl…
604c45a 
…e/Paddle into eager_math_op_div
@veyron95
update test
6aadc3c
@veyron95
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
e55b82a 
… eager_math_op_div
@veyron95
support div operator under cpp
5b5d326
@veyron95
fix scalar as input
e190777
@veyron95
Polish div logic, fix ci test
f6da2b3
@veyron95
fix errors
b530ca2
wanghuancoder
wanghuancoder approved these changes Sep 27, 2022
View reviewed changes
Copy link
Contributor

@wanghuancoder wanghuancoder left a comment

Choose a reason for hiding this comment

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

LGTM

@veyron95 veyron95 merged commit f20b361 into PaddlePaddle:develop Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wanghuancoder wanghuancoder wanghuancoder approved these changes

@chenwhql chenwhql chenwhql approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@veyron95 @chenwhql @wanghuancoder