You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/01/12 15:11:09 UTC
[GitHub] [tvm] luyaor opened a new issue #7263: [Bug] [Frontend] Matrix C of Gemm operator in ONNX could be optional
luyaor opened a new issue #7263:
URL: https://github.com/apache/tvm/issues/7263
## Description
When compiling following model with TVM, it will error on parsing. I think it is because matrix C for inputs could be optional by ONNX specification. Check here: https://github.com/onnx/onnx/blob/master/docs/Operators.md#gemm
The model(with ONNX as frontend) with error is as follows, check bug.onnx in [bug7.zip](https://github.com/apache/tvm/files/5802949/bug7.zip).
## Error Log
```
Traceback (most recent call last):
File "check.py", line 11, in <module>
mod, params = relay.frontend.from_onnx(onnx_model, {})
File "/Users/luyaor/Documents/tvm/python/tvm/relay/frontend/onnx.py", line 2806, in from_onnx
mod, params = g.from_onnx(graph, opset, freeze_params)
File "/Users/luyaor/Documents/tvm/python/tvm/relay/frontend/onnx.py", line 2613, in from_onnx
op = self._convert_operator(op_name, inputs, attr, opset)
File "/Users/luyaor/Documents/tvm/python/tvm/relay/frontend/onnx.py", line 2721, in _convert_operator
sym = convert_map[op_name](inputs, attrs, self._params)
File "/Users/luyaor/Documents/tvm/python/tvm/relay/frontend/onnx.py", line 510, in _impl_v1
assert len(inputs) == 3, "Gemm op take 3 inputs, {} given".format(len(inputs))
AssertionError: Gemm op take 3 inputs, 2 given
```
## How to reproduce
### Environment
Python3, with tvm, onnx
tvm version: [`c31e338`](https://github.com/apache/tvm/commit/c31e338d5f98a8e8c97286c5b93b20caee8be602) Wed Dec 9 14:52:58 2020 +0900
1. Download [bug7.zip](https://github.com/apache/tvm/files/5802949/bug7.zip)
2. Run `python check.py`.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] masahi closed issue #7263: [Bug] [Frontend] Matrix C of Gemm operator in ONNX could be optional
Posted by GitBox <gi...@apache.org>.
masahi closed issue #7263:
URL: https://github.com/apache/tvm/issues/7263
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org