Skip to content

restore caffe2 strides#12381

Closed
li-roy wants to merge 1 commit intopytorch:masterfrom
li-roy:export-D10220313
Closed

restore caffe2 strides#12381
li-roy wants to merge 1 commit intopytorch:masterfrom
li-roy:export-D10220313

Conversation

@li-roy
Copy link
Copy Markdown
Contributor

@li-roy li-roy commented Oct 5, 2018

Summary: The workflow passes after D10150834, so we can restore strides.

Differential Revision: D10220313

Copy link
Copy Markdown
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

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

That beautiful moment when you realize that it's easy :)

P.S., a note for the future: the threading fix should be put into a separate patch, to ease bisecters from the future (if the ordering there was actually important). For now, mentioning it in the commit message should be good enough.

@li-roy
Copy link
Copy Markdown
Contributor Author

li-roy commented Oct 9, 2018

@pytorchbot retest this please

Summary:
Pull Request resolved: pytorch#12381

The workflow passes after D10150834, so we can restore strides.

Reviewed By: ezyang

Differential Revision: D10220313

fbshipit-source-id: 9b943b82a5fec1717af60f4b17fa52eef0be36bd
zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 16, 2018
Summary:
Pull Request resolved: pytorch/pytorch#12381

The workflow passes after D10150834, so we can restore strides.

Reviewed By: ezyang

Differential Revision: D10220313

fbshipit-source-id: aaf9edebf4ff739cbe45b2d32e77918fce47ba34
@ezyang ezyang added the merged label Jun 26, 2019
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#12381

The workflow passes after D10150834, so we can restore strides.

Reviewed By: ezyang

Differential Revision: D10220313

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

2 participants