Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

Don't accept ONNX operators that aren't actually ONNX operators #63

Open
ezyang opened this issue Nov 4, 2017 · 0 comments
Open

Don't accept ONNX operators that aren't actually ONNX operators #63

ezyang opened this issue Nov 4, 2017 · 0 comments

Comments

@ezyang
Copy link
Collaborator

ezyang commented Nov 4, 2017

Right now, if you feed onnx-caffe2 something that happens to be a valid Caffe2 operator, but not an ONNX operator, onnx-caffe2 will silently accept it. That's bad! We do not want to reward bad behavior. And with the versioning update onnx/onnx#186 we can backdoor Caffe2 ops using a vendor extension, if you absolutely need to. So... tighten this up!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant