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

Support for tensorflowjs 3.17 -- maxNumElements attr in TensorListReserve dynamic list shoule be ignored

Open hwangdeyu opened this issue 2 years ago • 3 comments

5 tests are failed with latest tfjs vesion 3.17, which have skiped so far. Need to support it in the future.

https://github.com/onnx/tensorflow-onnx/blob/772dbe605e94880d4cbf8e9e1ed9ce9517b89c64/tests/test_cond.py#L121

https://github.com/onnx/tensorflow-onnx/blob/772dbe605e94880d4cbf8e9e1ed9ce9517b89c64/tests/test_loops.py#L69

https://github.com/onnx/tensorflow-onnx/blob/772dbe605e94880d4cbf8e9e1ed9ce9517b89c64/tests/test_loops.py#L162

https://github.com/onnx/tensorflow-onnx/blob/772dbe605e94880d4cbf8e9e1ed9ce9517b89c64/tests/test_loops.py#L186

https://github.com/onnx/tensorflow-onnx/blob/772dbe605e94880d4cbf8e9e1ed9ce9517b89c64/tests/test_loops.py#L224

hwangdeyu avatar May 18 '22 08:05 hwangdeyu

Created an issue https://github.com/tensorflow/tfjs/issues/6487.

hwangdeyu avatar Jun 02 '22 08:06 hwangdeyu

Created an issue tensorflow/tfjs#6487.

tfjs community fixed this issue by https://github.com/tensorflow/tfjs/pull/6523. Wait for a new release to support it.

hwangdeyu avatar Jun 17 '22 07:06 hwangdeyu

Mark related issue https://github.com/tensorflow/tfjs/issues/6748.

hwangdeyu avatar Aug 29 '22 09:08 hwangdeyu