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

Validate operator input counts #133

Open
robertknight opened this issue Apr 29, 2024 · 0 comments
Open

Validate operator input counts #133

robertknight opened this issue Apr 29, 2024 · 0 comments
Labels
tooling Tools for debugging / profiling etc.

Comments

@robertknight
Copy link
Owner

Model conversion doesn't currently check the number of inputs to each operator. This can lead to errors at runtime if it is different than what is expected. eg. In #132, the Reduce* ops expected a single input but were being passed two, as an attribute had been promoted to an input, which has been a fairly common occurrence in the evolution of the ONNX spec.

@robertknight robertknight added the tooling Tools for debugging / profiling etc. label Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tooling Tools for debugging / profiling etc.
Projects
None yet
Development

No branches or pull requests

1 participant