Skip to content

Conversation

@nairbv
Copy link
Collaborator

@nairbv nairbv commented Nov 5, 2018

update roll to behave as in numpy.roll when dimension to roll not specified.

@nairbv
Copy link
Collaborator Author

nairbv commented Nov 5, 2018

@pytorchbot retest this please

1 similar comment
@nairbv
Copy link
Collaborator Author

nairbv commented Nov 6, 2018

@pytorchbot retest this please

Copy link
Contributor

@gchanan gchanan left a comment

Choose a reason for hiding this comment

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

looks good, just nits.

Copy link
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.

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

zdevito pushed a commit to zdevito/ATen that referenced this pull request Nov 8, 2018
Summary:
update roll to behave as in numpy.roll when dimension to roll not specified.
Pull Request resolved: pytorch/pytorch#13588

Differential Revision: D12964295

Pulled By: nairbv

fbshipit-source-id: de9cdea1a937773033f081f8c1505a40e4e08bc1
@ezyang ezyang added the merged label Jun 25, 2019
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.

4 participants