Skip to content

TF-TRT test Binary op conversion in dynamic shape mode - #39785

Merged
tensorflow-copybara merged 1 commit into
tensorflow:masterfrom
tfeher:trt_binary_dynamic_shape
Jun 5, 2020
Merged

tensorflow-copybara merged 1 commit into
tensorflow:masterfrom
tfeher:trt_binary_dynamic_shape

Conversation

@tfeher

@tfeher tfeher commented May 22, 2020

Copy link
Copy Markdown
Contributor

This PR adds explicit batch and dynamic shape tests to the Binary op converter. Tagging @bixia1 for review.

@google-ml-butler google-ml-butler Bot added the size:L CL Change Size: Large label May 22, 2020
@google-ml-butler
google-ml-butler Bot requested a review from joker-eph May 22, 2020 09:41
@gbaned gbaned self-assigned this May 22, 2020
@gbaned gbaned added the comp:gpu:tensorrt Issues specific to TensorRT label May 22, 2020
@gbaned
gbaned requested a review from bixia1 May 22, 2020 10:41
@gbaned gbaned added the awaiting review Pull request awaiting review label May 27, 2020
@tfeher
tfeher force-pushed the trt_binary_dynamic_shape branch from 8d5d638 to 2533490 Compare May 28, 2020 16:22
@gbaned gbaned removed the awaiting review Pull request awaiting review label Jun 2, 2020

@tfeher tfeher left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @bixia1 for the review, I have addressed the issues.

@tfeher
tfeher force-pushed the trt_binary_dynamic_shape branch from 2533490 to cb3af46 Compare June 3, 2020 20:31
@tfeher
tfeher force-pushed the trt_binary_dynamic_shape branch from cb3af46 to b19e03c Compare June 4, 2020 14:38
@google-ml-butler google-ml-butler Bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jun 4, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jun 4, 2020
@tensorflow-copybara
tensorflow-copybara merged commit a8da58c into tensorflow:master Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes comp:gpu:tensorrt Issues specific to TensorRT ready to pull PR ready for merge process size:L CL Change Size: Large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants