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

Move OpProtoAndCheckerMaker to module operator #3511

Closed
JiayiFeng opened this issue Aug 15, 2017 · 0 comments · Fixed by #3512
Closed

Move OpProtoAndCheckerMaker to module operator #3511

JiayiFeng opened this issue Aug 15, 2017 · 0 comments · Fixed by #3512
Assignees
@JiayiFeng

Comments

@JiayiFeng
Copy link
Collaborator

JiayiFeng commented Aug 15, 2017 •
edited
Loading

OpProtoAndCheckerMaker is used to build a certain operator's OpProto and OpAttrChecker. All operators' OpMaker class is derived from it. it works very similar with the class OperatorBase in out design and is an important part of the whole operator definition. So it should be settled in Operator module, not OpRegistry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JiayiFeng JiayiFeng

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move OpRegistry functions to .cc file and move OpMaker to module operator JiayiFeng/Paddle
1 participant
@JiayiFeng