Skip to content

Commit

Permalink
Reenable test cases that are broken by TRT 4.0 since TF is now built …
Browse files Browse the repository at this point in the history
…against

TRT 5.0.

PiperOrigin-RevId: 228824151
  • Loading branch information
aaroey authored and tensorflower-gardener committed Jan 11, 2019
1 parent 03fb5da commit 764109a
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 40 deletions.
22 changes: 0 additions & 22 deletions tensorflow/contrib/tensorrt/test/batch_matmul_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,28 +86,6 @@ def ExpectedEnginesToRun(self, run_params):
"""Return the expected engines to run."""
return ["TRTEngineOp_1"]

def ShouldRunTest(self, run_params):
"""Whether to run the test."""
# TODO(aaroey): Trt library will fail like:
#
# ../builder/cudnnBuilder2.cpp:685:
# virtual std::vector<nvinfer1::query::Ports<
# nvinfer1::query::TensorRequirements>>
# nvinfer1::builder::Node::getSupportedFormats(
# const nvinfer1::query::Ports<nvinfer1::query::AbstractTensor>&,
# const nvinfer1::cudnn::HardwareContext&,
# nvinfer1::builder::Format::Type,
# const nvinfer1::builder::FormatTypeHack&) const:
# Assertion `sf' failed.
#
# To reproduce, run:
# bazel test -c opt --copt=-mavx \
# --test_arg=BatchMatMulTest.testTfTrt_ToolConversion_INT8_DynamicEngine \
# tensorflow/contrib/tensorrt:batch_matmul_test
#
# Investigate and fix it.
return not trt_test.IsQuantizationMode(run_params.precision_mode)


if __name__ == "__main__":
test.main()
6 changes: 0 additions & 6 deletions tensorflow/contrib/tensorrt/test/biasadd_matmul_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,12 +130,6 @@ def ExpectedEnginesToBuild(self, run_params):
"""Return the expected engines to build."""
return ["TRTEngineOp_0"]

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


if __name__ == "__main__":
test.main()
6 changes: 0 additions & 6 deletions tensorflow/contrib/tensorrt/test/identity_output_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,12 +69,6 @@ def ExpectedEnginesToBuild(self, run_params):
"""Return the expected engines to build."""
return ['TRTEngineOp_0']

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


if __name__ == '__main__':
test.main()
6 changes: 0 additions & 6 deletions tensorflow/contrib/tensorrt/test/rank_two_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,12 +80,6 @@ def ExpectedEnginesToBuild(self, run_params):
],
}

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


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

0 comments on commit 764109a

Please sign in to comment.