Bug in Python ONNX parser for Split node

[MatthewDaggitt]

Found another bug in the Python ONNX parser which documenting in case of failure to merge the C++ parser. The code doesn't query the num_outputs attribute, and treats incorrectly gets split as an attribute when it should be an input node. See https://github.com/onnx/onnx/blob/main/docs/Operators.md#Split for details.

It also unnecessarily recreates a whole set of new output variables.

Again, not planning to fix myself as the C++ implementation will be correct.

[akshaydr]

I am facing similar issue. Importing Split_18 from onnx.reference.ops._op_list fails. Is there a quick fix?

[MatthewDaggitt]

That doesn't sound like the same issue. Sounds like you have the wrong version of ONNX installed. Are 1.15.0 or higher?

[akshaydr]

You are correct. I had lower ONNX version. Thanks for a quick reply.