Skip to content

TF-TRT test FusedBatchNorm op converter - #40179

Merged
tensorflow-copybara merged 1 commit into
tensorflow:masterfrom
tfeher:trt_fusedbatchnorm_dynamic_shape
Jul 20, 2020
Merged

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

Conversation

@tfeher

@tfeher tfeher commented Jun 5, 2020

Copy link
Copy Markdown
Contributor

Add test for the FusedBatchNorm op converter in implicit batch, explicit batch and dynamic shape mode. Tagging @bixia1 for review.

@google-ml-butler google-ml-butler Bot added the size:L CL Change Size: Large label Jun 5, 2020
@google-ml-butler
google-ml-butler Bot requested a review from joker-eph June 5, 2020 06:50
@gbaned gbaned self-assigned this Jun 5, 2020
@gbaned gbaned added the comp:gpu:tensorrt Issues specific to TensorRT label Jun 5, 2020
@gbaned
gbaned requested a review from bixia1 June 5, 2020 08:58
@gbaned gbaned added the awaiting review Pull request awaiting review label Jun 8, 2020
@gbaned

gbaned commented Jul 3, 2020

Copy link
Copy Markdown
Contributor

@bixia1 Can you please review this PR ? Thanks!

@tfeher
tfeher force-pushed the trt_fusedbatchnorm_dynamic_shape branch from d91140e to 9eb20bb Compare July 6, 2020 22:01
@gbaned gbaned removed the awaiting review Pull request awaiting review label Jul 9, 2020
@gbaned

gbaned commented Jul 9, 2020

Copy link
Copy Markdown
Contributor

@tfeher Can you please check @bixia1's comments and keep us posted. Thanks!

@gbaned gbaned added awaiting review Pull request awaiting review stat:awaiting response Status - Awaiting response from author and removed awaiting review Pull request awaiting review labels Jul 9, 2020
@tfeher
tfeher force-pushed the trt_fusedbatchnorm_dynamic_shape branch from 9eb20bb to 6327c63 Compare July 11, 2020 21:16

@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.

@google-ml-butler google-ml-butler Bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jul 13, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jul 13, 2020
@gbaned gbaned removed the stat:awaiting response Status - Awaiting response from author label Jul 13, 2020

@bixia1 bixia1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry. I was in the process of merging this and realize that I need to make non-trivial changes to the comment for ConvertFusedBatchNorm. Need to get your agreement on this.

@rthadur rthadur removed the ready to pull PR ready for merge process label Jul 14, 2020
@gbaned

gbaned commented Jul 17, 2020

Copy link
Copy Markdown
Contributor

@tfeher Can you please check @bixia1's comments and keep us posted ? Thanks!

@gbaned gbaned added the stat:awaiting response Status - Awaiting response from author label Jul 17, 2020
@tfeher
tfeher force-pushed the trt_fusedbatchnorm_dynamic_shape branch from 6327c63 to 45591fa Compare July 20, 2020 14:58
@tfeher

tfeher commented Jul 20, 2020

Copy link
Copy Markdown
Contributor Author

Thanks @bixia1, I have made the requested changes.

@google-ml-butler google-ml-butler Bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Jul 20, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Jul 20, 2020
@tensorflow-copybara
tensorflow-copybara merged commit 90d28ce into tensorflow:master Jul 20, 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 stat:awaiting response Status - Awaiting response from author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants