Skip to content
This repository was archived by the owner on Oct 13, 2021. It is now read-only.

Add OnnxOperatorBuilder #112

Merged
merged 3 commits into from
Jun 10, 2019
Merged

Add OnnxOperatorBuilder #112

merged 3 commits into from
Jun 10, 2019

Conversation

jiafatom
Copy link
Collaborator

This is already tested in the private branch. Will add more tests later.

@jiafatom jiafatom requested a review from vinitra-zz June 10, 2019 20:57
Copy link

@vinitra-zz vinitra-zz left a comment

Choose a reason for hiding this comment

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

LGTM. If more converters decide to use this structure, we should eventually move this to onnxconverter-common!

@jiafatom jiafatom merged commit 20e461e into onnx:master Jun 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants