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

Clear extra attrs of elementwise op in OpMaker #45845

Merged
merged 7 commits into from
Sep 15, 2022

Conversation

zyfncg
Copy link
Contributor

@zyfncg zyfncg commented Sep 7, 2022

PR types

Others

PR changes

OPs

Describe

elementwise相关算子OpMaker的Extra参数清理。

相关PR:
第一批算子OpMaker清理: PR45613
第二批算子OpMaker清理: PR45684
第三批算子OpMaker清理: PR45758
matmul_v2算子OpMaker清理: PR45708
activation算子OpMaker清理: PR45772
reduce算子OpMaker清理: PR45786

@paddle-bot
Copy link

paddle-bot bot commented Sep 7, 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.

@zyfncg zyfncg merged commit b26efe0 into PaddlePaddle:develop Sep 15, 2022
@zyfncg zyfncg deleted the clear_elementwise_opmaker branch September 15, 2022 06:59
zyfncg added a commit to zyfncg/Paddle that referenced this pull request Sep 19, 2022
* clear extra attrs of elementwise op in opmaker

* fix op_debug_string_test

* fix bug of grad_add

* fix sort of runtime attrs
phlrain pushed a commit that referenced this pull request Sep 27, 2022
… 46060) (#46218)

* Clear extra attrs of elementwise op in OpMaker (#45845)

* clear extra attrs of elementwise op in opmaker

* fix op_debug_string_test

* fix bug of grad_add

* fix sort of runtime attrs

* Clear extra attrs of scale in OpMaker (#45984)

* clear extra attr of scale in opmaker

* fix sum bug

* fix merge conflict

* fix minus

* Clear extra attributes of some Op in OpMaker (Part4) (#46060)

* clear extra attr of some ops in opmaker

* revert clear use_cudnn for pool

* fix test_operator_desc

* fix Attr interface of OperatorBase

* fix code stype
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.

2 participants