Skip to content

Remove THTensor_isSize.#10146

Closed
gchanan wants to merge 2 commits intopytorch:masterfrom
gchanan:remove_isSize
Closed

Remove THTensor_isSize.#10146
gchanan wants to merge 2 commits intopytorch:masterfrom
gchanan:remove_isSize

Conversation

@gchanan
Copy link
Contributor

@gchanan gchanan commented Aug 1, 2018

This is part of the process of removing THLongStorage to represent sizes/strides.

This is part of the process of removing THLongStorage to represent sizes/strides.
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.

gchanan has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

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

very nice

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.

gchanan has imported 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 Aug 2, 2018
Summary:
This is part of the process of removing THLongStorage to represent sizes/strides.
Pull Request resolved: pytorch/pytorch#10146

Differential Revision: D9126611

Pulled By: gchanan

fbshipit-source-id: b0d995a4c51dfd54bf76dcfee9a69f37f9d01652
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
This is part of the process of removing THLongStorage to represent sizes/strides.
Pull Request resolved: pytorch#10146

Differential Revision: D9126611

Pulled By: gchanan

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

3 participants