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 condition op in OpMaker #46150

Merged
merged 2 commits into from
Sep 20, 2022

Conversation

zyfncg
Copy link
Contributor

@zyfncg zyfncg commented Sep 16, 2022

PR types

Others

PR changes

OPs

Describe

whileconditional_block算子OpMaker清理

相关PR:
第一批算子OpMaker清理: PR45613
第二批算子OpMaker清理: PR45684
第三批算子OpMaker清理: PR45758
第四批算子OpMaker清理: PR 46060
matmul_v2算子OpMaker清理: PR45708
activation算子OpMaker清理: PR45772
reduce算子OpMaker清理: PR45786
elementwise算子OpMaker清理: PR45845
scale算子OpMaker清理: PR45984

@paddle-bot
Copy link

paddle-bot bot commented Sep 16, 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 f65a61a into PaddlePaddle:develop Sep 20, 2022
@zyfncg zyfncg deleted the clear_condition_opmaker branch September 20, 2022 02:31
zyfncg added a commit to zyfncg/Paddle that referenced this pull request Sep 27, 2022
phlrain pushed a commit that referenced this pull request Sep 28, 2022
#46418, #46451, #46457) (#46553)

* 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

* clear extra attrs of condition op in opmaker (#46150)

* Clear extra attrs of lookup_table_v2 in OpMaker (#46321)

* clear extra attrs of look_up_table_v2 in opmaker

* fix bug

* clear extra attrs of quantize op in opmaker (#46418)

* delete repeated item

* clear extra attrs of distribute op in opmaker (#46451)

* clear extra atts of sequence_softmax in opmaker (#46457)
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.

3 participants