tensorflow-onnx icon indicating copy to clipboard operation
tensorflow-onnx copied to clipboard

Unsupported ops: TensorScatterAdd, TensorListConcatV2, Rank

Open igalkr opened this issue 3 years ago • 7 comments

Hi, are the above ops to be supported, or am I running the command incorrectly? python -m tf2onnx.convert --saved-model saved_model --output model.onnx -v --target tensorrt --tag serve

igalkr avatar Nov 18 '21 17:11 igalkr

I am having a very confusing problem with tf2onnx giving me TensorScatterAdd nodes. The tf code I am trying to convert to onnx used tf.tensor_scatter_nd_add. I replaced this with tf.scatter_nd and tf.math.add in the hopes that tf2onnx would stop giving me this TensorScatterAdd node, but I am still getting it. It is almost as if tf2onnx is fusing these two operations for me.

More generally, I am confused why the tf2onnx parser is giving me a node that is not supported by onnx. I would love an explanation of what is going on here!

alexarmbr avatar Nov 19 '21 13:11 alexarmbr

Yes, TensorScatterAdd, TensorListConcatV2 are not mapped - we'd need to add it.

guschmue avatar Nov 19 '21 17:11 guschmue

What should I do in the meantime? I really need the onnx converted file.

igalkr avatar Nov 25 '21 14:11 igalkr

What should I do in the meantime? I really need the onnx converted file.

Could you please share the TF code of your model so that we can validate it once there is a fix?

fatcat-z avatar Dec 06 '21 05:12 fatcat-z

Hi, I had to get in contact with our management, we cannot share the code at the moment.

igalkr avatar Jan 02 '22 11:01 igalkr

:+1: I'd really like to see TensorScatterAdd be added to tf2onnx as well.

r-remus avatar Mar 25 '22 12:03 r-remus

Hi @r-remus @igalkr, thanks for your comments! TensorScatterAdd has added with this PR Add TensorScatterAdd op for opset 16. You can try to convert it using latest main branch code or waitting for the next tf2onnx release.

hwangdeyu avatar May 31 '22 08:05 hwangdeyu

It's been over 2 months, so closing this. Feel free to open a new one if the issue still exists.

fatcat-z avatar Oct 11 '22 04:10 fatcat-z

Hi, I am getting the following error with opset=18:

Tensorflow op [TensorListConcatV2_4: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2_3: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2_2: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2_1: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2: TensorListConcatV2] is not supported Unsupported ops: Counter({'TensorListConcatV2': 5})

could you please help me?

k-maheshkumar avatar Nov 10 '23 15:11 k-maheshkumar

Hi, I am getting the following error with opset=18:

Tensorflow op [TensorListConcatV2_4: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2_3: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2_2: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2_1: TensorListConcatV2] is not supported Tensorflow op [TensorListConcatV2: TensorListConcatV2] is not supported Unsupported ops: Counter({'TensorListConcatV2': 5})

could you please help me?

This op is not supported yet. Please feel free to open a new issue with some details, like code or model, for this requirement so that we can put it into the plan.

fatcat-z avatar Dec 13 '23 02:12 fatcat-z