Skip to content

Split storage from tensor#10053

Closed
jerryzh168 wants to merge 1 commit intopytorch:masterfrom
jerryzh168:export-D9076734
Closed

Split storage from tensor#10053
jerryzh168 wants to merge 1 commit intopytorch:masterfrom
jerryzh168:export-D9076734

Conversation

@jerryzh168
Copy link
Contributor

Summary:
Tensor in Pytorch 1.0 will have
Tensor -> TensorImpl -> Storage -> StorageImpl
In this diff we split Storage from Tensor in order to align with this design.
We'll have Tensor -> Storage -> StorageImpl after this diff

Differential Revision: D9076734

@jerryzh168
Copy link
Contributor Author

@pytorchbot retest this please

1 similar comment
@jerryzh168
Copy link
Contributor Author

@pytorchbot retest this please

Summary:
Pull Request resolved: pytorch#10053

Tensor in Pytorch 1.0 will have
Tensor -> TensorImpl -> Storage -> StorageImpl
In this diff we split Storage from Tensor in order to align with this design.
We'll have Tensor -> Storage -> StorageImpl after this diff

Reviewed By: dzhulgakov

Differential Revision: D9076734

fbshipit-source-id: a9bef96bec6b2e570434c139951a266d4ecaf378
jerryzh168 added a commit to jerryzh168/pytorch that referenced this pull request Aug 22, 2018
Summary:
Pull Request resolved: pytorch#10053

Tensor in Pytorch 1.0 will have
Tensor -> TensorImpl -> Storage -> StorageImpl
In this diff we split Storage from Tensor in order to align with this design.
We'll have Tensor -> Storage -> StorageImpl after this diff

Reviewed By: ezyang

Differential Revision: D9384781

fbshipit-source-id: ee3a5a8641436743320f62a346971f407fc51c0c
facebook-github-bot pushed a commit that referenced this pull request Aug 22, 2018
Summary:
Pull Request resolved: #10053

Tensor in Pytorch 1.0 will have
Tensor -> TensorImpl -> Storage -> StorageImpl
In this diff we split Storage from Tensor in order to align with this design.
We'll have Tensor -> Storage -> StorageImpl after this diff

Reviewed By: ezyang

Differential Revision: D9384781

fbshipit-source-id: 40ded2437715a3a2cc888ef28cbca9a25b1d5350
PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
Pull Request resolved: pytorch#10053

Tensor in Pytorch 1.0 will have
Tensor -> TensorImpl -> Storage -> StorageImpl
In this diff we split Storage from Tensor in order to align with this design.
We'll have Tensor -> Storage -> StorageImpl after this diff

Reviewed By: ezyang

Differential Revision: D9384781

fbshipit-source-id: 40ded2437715a3a2cc888ef28cbca9a25b1d5350
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants