u99127 commented on a change in pull request #6774:
URL: https://github.com/apache/incubator-tvm/pull/6774#discussion_r513030165
##########
File path: tests/python/frontend/tflite/test_forward.py
##########
@@ -2548,14 +2556,17 @@ def test_forward_padv2():
np.array([2], dtype=np.float32),
]
)
- _test_padv2(
- [
- np.arange(0, 256, dtype=np.uint8).reshape((1, 256)),
- np.array([[1, 1], [2, 2]], dtype=np.int32),
- np.array([2], dtype=np.uint8),
- ],
- quantized=True,
- )
+ # NOTE: In recent version, there is a bug in Tensorflow package for this
scenario.
+ # Hence, it is disabled temporarily for latest TF version.
Review comment:
Please say
In versions > 2.1.0 instead of "recent version".
If there's a reference to " a bug in tensorflow package" can you please link
to the bug in the tensorflow package ? Given our processes for issues are less
defined, I would suggest atleast an issue in our issue tracker to remove this
on a later date as a code cleanup once the tensorflow bug is fixed.
##########
File path: tests/python/frontend/tflite/test_forward.py
##########
@@ -2565,14 +2576,17 @@ def test_forward_padv2():
np.float32(2),
]
)
- _test_padv2(
- [
- np.arange(0, 256, dtype=np.uint8).reshape((1, 256)),
- np.array([[1, 1], [2, 2]], dtype=np.int32),
- np.uint8(10),
- ],
- quantized=True,
- )
+ # NOTE: In recent version, there is a bug in Tensorflow package for this
scenario.
+ # Hence, it is disabled temporarily for latest TF version.
Review comment:
Same comment as above.
----------------------------------------------------------------
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:
[email protected]
