Skip to content

Conversation

@nairbv
Copy link
Collaborator

@nairbv nairbv commented Nov 12, 2018

No description provided.

@nairbv
Copy link
Collaborator Author

nairbv commented Nov 13, 2018

@pytorch retest this please

getting "RuntimeError: Address already in use"

@nairbv
Copy link
Collaborator Author

nairbv commented Nov 14, 2018

@pytorchbot retest this please

still getting RuntimeError: Address already in use

@nairbv nairbv self-assigned this Nov 14, 2018
@nairbv nairbv requested a review from gchanan November 14, 2018 22:05
@gchanan
Copy link
Contributor

gchanan commented Nov 16, 2018

should the documentation be updated?

@nairbv
Copy link
Collaborator Author

nairbv commented Nov 16, 2018

should the documentation be updated?

ah, the existing documentation mostly covered this case "(int or tuple of ints)," but I updated to clarify this use case a bit.

@nairbv
Copy link
Collaborator Author

nairbv commented Nov 16, 2018

numpy.roll also allows an int shift to be used with a tuple axis, applying the same shift to all dimensions. This revision doesn't support that case, requiring shifts.size==dims.size unless dims==None

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.

documentation update?

auto tail_shifts = shifts.slice(1);
auto tail_dims = dims.slice(1);
auto first_dim_rolled = roll(self, shifts[0], dims[0]);
return roll(first_dim_rolled, tail_shifts, tail_dims);

This comment was marked as off-topic.

This comment was marked as off-topic.

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 28, 2018
Summary: Pull Request resolved: pytorch/pytorch#13874

Differential Revision: D13223669

Pulled By: nairbv

fbshipit-source-id: 1678d52529c326fa4a0614d0994b1820ad12bc04
@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.

5 participants