Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[v1.5.x] add deconv in TRT subgraph (#15666) #16043

Merged
merged 1 commit into from
Aug 31, 2019
Merged

[v1.5.x] add deconv in TRT subgraph (#15666) #16043

merged 1 commit into from
Aug 31, 2019

Conversation

KellenSunderland
Copy link
Contributor

Description

This PR add deconv layer to TRT subgraph

Checklist

Essentials

  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage:
    • added a unit test in tests/python/tensorrt/test_tensorrt_deconv
  • Code is well-documented:

Changes

  • added deconv in subgraph conversion

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

@KellenSunderland KellenSunderland mentioned this pull request Aug 30, 2019
Closed
@KellenSunderland KellenSunderland changed the title add deconv in TRT subgraph (#15666) [v1.5.x] add deconv in TRT subgraph (#15666) Aug 30, 2019
@TaoLv
Copy link
Member

TaoLv commented Aug 30, 2019

Thank you @KellenSunderland. Please take a look at the CI failure.

@KellenSunderland
Copy link
Contributor Author

@TaoLv Looks like a worker failed to report the status of a build. Rebuilding unix-cpu.

@karan6181
Copy link
Contributor

@mxnet-label-bot add [Operator, C++, pr-awaiting-response]

@marcoabreu marcoabreu added C++ Related to C++ Operator pr-awaiting-response PR is reviewed and waiting for contributor to respond labels Aug 30, 2019
@KellenSunderland KellenSunderland removed the pr-awaiting-response PR is reviewed and waiting for contributor to respond label Aug 31, 2019
@KellenSunderland KellenSunderland merged commit afe4a71 into apache:v1.5.x Aug 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@TaoLv TaoLv TaoLv approved these changes

Assignees
No one assigned
Labels
C++ Related to C++ Operator
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@KellenSunderland @TaoLv @karan6181 @marcoabreu @haohuanw