Skip to content

Conversation

@ezyang
Copy link
Contributor

@ezyang ezyang commented Jun 12, 2019

Stack from ghstack:

Fixes #20756

Signed-off-by: Edward Z. Yang [email protected]

Differential Revision: D15807402

@pytorchbot pytorchbot added the module: nn Related to torch.nn label Jun 12, 2019
@ezyang
Copy link
Contributor Author

ezyang commented Jun 12, 2019

I have a test case but it is blocked on getting access to our AWS bucket so I can upload the test data.

@ezyang ezyang requested a review from pietern June 12, 2019 16:11
Copy link
Contributor

@pietern pietern left a comment

Choose a reason for hiding this comment

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

LGTM! If the none constant is defined elsewhere (perhaps even as default) then you could use that?

ezyang added 3 commits June 13, 2019 10:24
…pickles."

Define __setstate__ on _ConvNd to handle pre-padding_mode pickles.

Fixes #20756

Signed-off-by: Edward Z. Yang <[email protected]>

gh-metadata: pytorch pytorch 21687 gh/ezyang/167/head
…pickles."

Define __setstate__ on _ConvNd to handle pre-padding_mode pickles.

Fixes #20756

Signed-off-by: Edward Z. Yang <[email protected]>

gh-metadata: pytorch pytorch 21687 gh/ezyang/167/head
…handle pre-padding_mode pickles."

Define __setstate__ on _ConvNd to handle pre-padding_mode pickles.

Fixes #20756

Signed-off-by: Edward Z. Yang <[email protected]>

gh-metadata: pytorch pytorch 21687 gh/ezyang/167/head
@pytorchbot pytorchbot added the module: infra Relates to CI infrastructure label Jun 13, 2019
ezyang added 2 commits June 14, 2019 08:54
…handle pre-padding_mode pickles."

Define __setstate__ on _ConvNd to handle pre-padding_mode pickles.

Fixes #20756

Signed-off-by: Edward Z. Yang <[email protected]>

gh-metadata: pytorch pytorch 21687 gh/ezyang/167/head
…pickles."

Define __setstate__ on _ConvNd to handle pre-padding_mode pickles.

Fixes #20756

Signed-off-by: Edward Z. Yang <[email protected]>

gh-metadata: pytorch pytorch 21687 gh/ezyang/167/head
@zou3519 zou3519 deleted the gh/ezyang/167/head branch June 14, 2019 18:03
@facebook-github-bot
Copy link
Contributor

@ezyang merged this pull request in 029a968.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: infra Relates to CI infrastructure module: nn Related to torch.nn

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants