Skip to content

[CustomOp] Add new paddle custom op so#31141

Merged
chenwhql merged 4 commits intoPaddlePaddle:developfrom
chenwhql:extension/add_custom_op_so
Feb 24, 2021
Merged

[CustomOp] Add new paddle custom op so#31141
chenwhql merged 4 commits intoPaddlePaddle:developfrom
chenwhql:extension/add_custom_op_so

Conversation

@chenwhql
Copy link
Copy Markdown
Contributor

@chenwhql chenwhql commented Feb 23, 2021

PR types

Function optimization

PR changes

Others

Describe

[CustomOp] Add new paddle custom op so

Because the original paddle_framework.so constains some unneeded targets, such as executor, recurrent_op etc, which will cause inference loading so failed, so we add a new custom_op.so for new custom op module, and keep the original paddle_framework.so unchanged.

we will remove paddle_framework.so in the future.

@paddle-bot-old
Copy link
Copy Markdown

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

Aurelius84
Aurelius84 previously approved these changes Feb 24, 2021
JiabinYang
JiabinYang previously approved these changes Feb 24, 2021
Copy link
Copy Markdown
Contributor

@JiabinYang JiabinYang left a comment

Choose a reason for hiding this comment

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

LGTM

Shixiaowei02
Shixiaowei02 previously approved these changes Feb 24, 2021
chalsliu
chalsliu previously approved these changes Feb 24, 2021
Copy link
Copy Markdown
Contributor

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

LGTM

@chenwhql chenwhql merged commit 1ce96fa into PaddlePaddle:develop Feb 24, 2021
chenwhql added a commit to chenwhql/Paddle that referenced this pull request Feb 26, 2021
* add new custom op so

* fix use new method error

* fix test failed
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.

6 participants