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.

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 tanh_triple_grad composite logic #56072

Merged
merged 3 commits into from Aug 10, 2023
Merged

add tanh_triple_grad composite logic #56072

merged 3 commits into from Aug 10, 2023

Conversation

lxd-cumt
Copy link
Contributor

@lxd-cumt lxd-cumt commented Aug 8, 2023

PR types

Others

PR changes

Others

Description

decompose tanh_triple_grad, add tanh_triple_grad into prim_white_list, and fix bugs in TanhTripleGradKernel
Pcard-66975

@paddle-bot
Copy link

paddle-bot bot commented Aug 8, 2023

你的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.

@lxd-cumt lxd-cumt changed the title Tanhtriplegrad add add tanh_triple_grad composite logic Aug 8, 2023
Copy link
Contributor

@cxxly cxxly left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -72,7 +72,7 @@
"subtract_double_grad",
"add_triple_grad",
"silu_double_grad",
"tanh_double_grad",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why delete tanh_double_grad?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When decomposing tanh_triple_grad, the tanh_grad and tanh_double_grad can use fused kernels for better performance, thus remove tanh_double_grad from prim_white_list to prevent decomposing tanh_double_grad redundantly.

Copy link
Contributor

@zyfncg zyfncg left a comment

Choose a reason for hiding this comment

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

LGTM

@cxxly cxxly merged commit 7c4a355 into PaddlePaddle:develop Aug 10, 2023
27 checks passed
@lxd-cumt lxd-cumt deleted the tanhtriplegrad_add branch August 10, 2023 09:35
lxd-cumt added a commit to lxd-cumt/Paddle that referenced this pull request Aug 15, 2023
* decompose tanh_triple_grad and add it into prim_white_list test=develop

* fix TanhTripleGradKernel bugs test=develop

* decompose tanh_triple_grad test=develop
lxd-cumt added a commit to lxd-cumt/Paddle that referenced this pull request Nov 3, 2023
* decompose tanh_triple_grad and add it into prim_white_list test=develop

* fix TanhTripleGradKernel bugs test=develop

* decompose tanh_triple_grad test=develop
lxd-cumt added a commit to lxd-cumt/Paddle that referenced this pull request Nov 3, 2023
* decompose tanh_triple_grad and add it into prim_white_list test=develop

* fix TanhTripleGradKernel bugs test=develop

* decompose tanh_triple_grad test=develop
XiaoguangHu01 pushed a commit that referenced this pull request Nov 6, 2023
* decompose tanh_triple_grad and add it into prim_white_list test=develop

* fix TanhTripleGradKernel bugs test=develop

* decompose tanh_triple_grad test=develop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants