Skip to content

[TFTRT - Dynamic Shape Phase 3] Add Dynamic Shape Testing for ConvertAddN and DebugString templated for nested numerical vectors - #46675

Merged
copybara-service[bot] merged 1 commit into
tensorflow:masterfrom
DEKHTIARJonathan:jdekhtiar/dynamic_shape/ConvertAddN
Feb 5, 2021
Merged

copybara-service[bot] merged 1 commit into
tensorflow:masterfrom
DEKHTIARJonathan:jdekhtiar/dynamic_shape/ConvertAddN

Conversation

@DEKHTIARJonathan

Copy link
Copy Markdown
Contributor

@bixia1 @tfeher for review

Feature Tracker: #45481

@google-ml-butler google-ml-butler Bot added the size:M CL Change Size: Medium label Jan 25, 2021
@google-ml-butler
google-ml-butler Bot requested a review from joker-eph January 25, 2021 22:43
@google-cla google-cla Bot added the cla: yes label Jan 25, 2021
@gbaned gbaned self-assigned this Jan 26, 2021
@gbaned gbaned added the comp:gpu:tensorrt Issues specific to TensorRT label Jan 26, 2021
@gbaned
gbaned requested a review from smit-hinsu January 26, 2021 03:28
@bixia1
bixia1 self-requested a review January 26, 2021 17:20
@smit-hinsu
smit-hinsu removed their request for review January 27, 2021 00:44
@DEKHTIARJonathan

Copy link
Copy Markdown
Contributor Author

@bixia1 I think I covered all your reviews. Can you please check ;)

@DEKHTIARJonathan

Copy link
Copy Markdown
Contributor Author

@bixia1 I updated the PR. Can you please elaborate your comment : #46675 (comment)

I didn't get what you mean here

@DEKHTIARJonathan DEKHTIARJonathan changed the title [TFTRT - Dynamic Shape Phase 3] Add Dynamic Shape Testing for ConvertAddN [TFTRT - Dynamic Shape Phase 3] Add Dynamic Shape Testing for ConvertAddN and DebugString templated for nested numerical vectors Feb 2, 2021

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

Thanks! Just a typo in the comment and I will approve it after this.

@DEKHTIARJonathan

Copy link
Copy Markdown
Contributor Author

We are all good this time @bixia1 ;)

@google-ml-butler google-ml-butler Bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels Feb 2, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Feb 2, 2021
@rthadur rthadur added ready to pull PR ready for merge process and removed ready to pull PR ready for merge process labels Feb 3, 2021
@DEKHTIARJonathan

Copy link
Copy Markdown
Contributor Author

@bixia1 could you let me know what is the issue with this PR ? That way we can move forward ?

@bixia1

bixia1 commented Feb 5, 2021

Copy link
Copy Markdown
Contributor

It currently stuck at the merge process, likely due to some irrelevant test failure.

@copybara-service
copybara-service Bot merged commit 7ca8021 into tensorflow:master Feb 5, 2021
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:M CL Change Size: Medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants