Skip to content

Fix convtranspose3d output_size calculation#12952

Closed
zou3519 wants to merge 1 commit intopytorch:masterfrom
zou3519:convtranspose-fix
Closed

Fix convtranspose3d output_size calculation#12952
zou3519 wants to merge 1 commit intopytorch:masterfrom
zou3519:convtranspose-fix

Conversation

@zou3519
Copy link
Copy Markdown
Contributor

@zou3519 zou3519 commented Oct 22, 2018

Closes #2119.

There was a small bug where the output_size got sliced with [-2:]
where we really meant to slice it as [2:] (to remove the batch and
channel dimensions).

Added a new test for this.

Closes pytorch#2119.

There was a small bug where the output_size got sliced with `[-2:]`
where we really meant to slice it as `[2:]` (to remove the batch and
channel dimensions).

Added a new test for this.
@zou3519 zou3519 requested a review from soumith October 22, 2018 21:19
Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

zou3519 is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@ezyang ezyang added the merged label Jun 25, 2019
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Closes pytorch#2119.

There was a small bug where the output_size got sliced with `[-2:]`
where we really meant to slice it as `[2:]` (to remove the batch and
channel dimensions).

Added a new test for this.
Pull Request resolved: pytorch#12952

Differential Revision: D10510678

Pulled By: zou3519

fbshipit-source-id: 4c04a5007fc6d002e1806d6fe981b43d33d6a4f2
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.

ConvTranspose3d with output_size raises ValueError

4 participants