jwfromm edited a comment on pull request #8867: URL: https://github.com/apache/tvm/pull/8867#issuecomment-907659234
Onnx has strict conventions around input ordering. Any optional input that isnt used, but has other inputs after it that are used, must be provided and have its name set to an empty string. If this convention isnt followed it would be impossible to tell which input is which. This is the key point that @mbrookhart and I weren't aware of when we tried to use input names. See [here ](https://github.com/onnx/onnx/blob/master/docs/IR.md)for more info. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
