Skip to content

Commit 764109a

Browse files
aaroeytensorflower-gardener
authored andcommitted
Reenable test cases that are broken by TRT 4.0 since TF is now built against
TRT 5.0. PiperOrigin-RevId: 228824151
1 parent 03fb5da commit 764109a

File tree

4 files changed

+0
-40
lines changed

4 files changed

+0
-40
lines changed

tensorflow/contrib/tensorrt/test/batch_matmul_test.py

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -86,28 +86,6 @@ def ExpectedEnginesToRun(self, run_params):
8686
"""Return the expected engines to run."""
8787
return ["TRTEngineOp_1"]
8888

89-
def ShouldRunTest(self, run_params):
90-
"""Whether to run the test."""
91-
# TODO(aaroey): Trt library will fail like:
92-
#
93-
# ../builder/cudnnBuilder2.cpp:685:
94-
# virtual std::vector<nvinfer1::query::Ports<
95-
# nvinfer1::query::TensorRequirements>>
96-
# nvinfer1::builder::Node::getSupportedFormats(
97-
# const nvinfer1::query::Ports<nvinfer1::query::AbstractTensor>&,
98-
# const nvinfer1::cudnn::HardwareContext&,
99-
# nvinfer1::builder::Format::Type,
100-
# const nvinfer1::builder::FormatTypeHack&) const:
101-
# Assertion `sf' failed.
102-
#
103-
# To reproduce, run:
104-
# bazel test -c opt --copt=-mavx \
105-
# --test_arg=BatchMatMulTest.testTfTrt_ToolConversion_INT8_DynamicEngine \
106-
# tensorflow/contrib/tensorrt:batch_matmul_test
107-
#
108-
# Investigate and fix it.
109-
return not trt_test.IsQuantizationMode(run_params.precision_mode)
110-
11189

11290
if __name__ == "__main__":
11391
test.main()

tensorflow/contrib/tensorrt/test/biasadd_matmul_test.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -130,12 +130,6 @@ def ExpectedEnginesToBuild(self, run_params):
130130
"""Return the expected engines to build."""
131131
return ["TRTEngineOp_0"]
132132

133-
def ShouldRunTest(self, run_params):
134-
"""Whether to run the test."""
135-
# TODO(aaroey): Trt 4.0 forbids conversion for tensors with rank <3 in int8
136-
# mode, which is a bug. Re-enable this when trt library is fixed.
137-
return not trt_test.IsQuantizationMode(run_params.precision_mode)
138-
139133

140134
if __name__ == "__main__":
141135
test.main()

tensorflow/contrib/tensorrt/test/identity_output_test.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -69,12 +69,6 @@ def ExpectedEnginesToBuild(self, run_params):
6969
"""Return the expected engines to build."""
7070
return ['TRTEngineOp_0']
7171

72-
def ShouldRunTest(self, run_params):
73-
"""Whether to run the test."""
74-
# TODO(aaroey): Trt 4.0 forbids conversion for tensors with rank <3 in int8
75-
# mode, which is a bug. Re-enable this when trt library is fixed.
76-
return not trt_test.IsQuantizationMode(run_params.precision_mode)
77-
7872

7973
if __name__ == '__main__':
8074
test.main()

tensorflow/contrib/tensorrt/test/rank_two_test.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -80,12 +80,6 @@ def ExpectedEnginesToBuild(self, run_params):
8080
],
8181
}
8282

83-
def ShouldRunTest(self, run_params):
84-
"""Whether to run the test."""
85-
# TODO(aaroey): Trt 4.0 forbids conversion for tensors with rank <3 in int8
86-
# mode, which is a bug. Re-enable this when trt library is fixed.
87-
return not trt_test.IsQuantizationMode(run_params.precision_mode)
88-
8983

9084
if __name__ == "__main__":
9185
test.main()

0 commit comments

Comments
 (0)