Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Explicit padding for const shape ConvTranspose #1513

Merged

Conversation

calvinmccarter-at-lightmatter
Copy link
Contributor

This creates a ConvTranspose node with constant shape using explicitly-defined padding rather than output_shape-based padding (issue #1501).

Signed-off-by: Calvin McCarter <calvin@lightmatter.co>
Signed-off-by: Calvin McCarter <calvin@lightmatter.co>
Signed-off-by: Calvin McCarter <calvin@lightmatter.co>
@calvinmccarter-at-lightmatter calvinmccarter-at-lightmatter marked this pull request as draft May 12, 2021 09:24
Signed-off-by: Calvin McCarter <calvin@lightmatter.co>
Signed-off-by: Calvin McCarter <calvin@lightmatter.co>
@lgtm-com
Copy link

lgtm-com bot commented May 12, 2021

This pull request introduces 1 alert when merging ab23dac into c7231d5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@calvinmccarter-at-lightmatter calvinmccarter-at-lightmatter marked this pull request as ready for review May 12, 2021 11:12
@lgtm-com
Copy link

lgtm-com bot commented May 12, 2021

This pull request introduces 1 alert when merging 2b4121f into b603705 - view on LGTM.com

new alerts:

  • 1 for Unused local variable

tf2onnx/onnx_opset/nn.py Outdated Show resolved Hide resolved
Signed-off-by: Calvin McCarter <calvin@lightmatter.co>
…-lightmatter/tensorflow-onnx into conv-transpose-padding
@guschmue guschmue merged commit 620b952 into onnx:master May 13, 2021
zerollzeng pushed a commit to zerollzeng/tensorflow-onnx that referenced this pull request May 16, 2021
* explicit padding for const shape ConvTranspose

Signed-off-by: Calvin McCarter <calvin@lightmatter.co>

* pylint fix

Signed-off-by: Calvin McCarter <calvin@lightmatter.co>

* test_backend graph_validator fix

Signed-off-by: Calvin McCarter <calvin@lightmatter.co>

* keep output_shape if input_dims unknown

Signed-off-by: Calvin McCarter <calvin@lightmatter.co>

* cleanup

Signed-off-by: Calvin McCarter <calvin@lightmatter.co>

* remove unused variable

Signed-off-by: Calvin McCarter <calvin@lightmatter.co>

Co-authored-by: TomWildenhain-Microsoft <67606533+TomWildenhain-Microsoft@users.noreply.github.com>
Co-authored-by: Guenther Schmuelling <guschmue@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: explicitly-defined padding in ConvTranspose
3 participants