Skip to content

[pt1][tensor] Remove ExtractDeviceOption from StaticContext#12304

Closed
jerryzh168 wants to merge 54 commits intomasterfrom
export-D10069839
Closed

[pt1][tensor] Remove ExtractDeviceOption from StaticContext#12304
jerryzh168 wants to merge 54 commits intomasterfrom
export-D10069839

Conversation

@jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Oct 3, 2018

Stack:
    :black_circle:  #12304 [pt1][tensor] Remove ExtractDeviceOption from StaticContext  💚
    :white_circle:  #12305 [pt1][tensor] Remove StaticContext  💚

  • make ExtractDeviceOption to be a free function.
  • Add a Strorage(at::Device) constructor in order to preserve the device_id.

Differential Revision: D10069839

Differential Revision: D9779821
Differential Version: 57806436
Differential Revision: D9779821
Differential Version: 58572220
Differential Revision: D9779821
Differential Version: 58709105
Differential Revision: D9779821
Differential Version: 58713403
Differential Revision: D9779821
Differential Version: 58748651
Differential Revision: D9889990
Differential Version: 58748485
Differential Revision: D9889990
Differential Version: 58751934
Differential Revision: D9889990
Differential Version: 58799279
Differential Revision: D10033396
Differential Version: 58815881
Differential Revision: D9889990
Differential Version: 58825947
Differential Revision: D10033396
Differential Version: 58835275
Differential Revision: D9889990
Differential Version: 58856304
Differential Revision: D10033396
Differential Version: 58856963
Differential Revision: D9889990
Differential Version: 58858304
Differential Revision: D9889990
Differential Version: 58875107
Differential Revision: D9889990
Differential Version: 58881242
Differential Revision: D10001033
Differential Version: 58937972
Differential Revision: D10022853
Differential Version: 58947676
Differential Revision: D10022853
Differential Version: 58951172
Differential Revision: D10033396
Differential Version: 59085834
Differential Revision: D10033396
Differential Version: 59088508
Differential Revision: D10033396
Differential Version: 59091126
Differential Revision: D9889990
Differential Version: 59110247
Differential Revision: D10033396
Differential Version: 59227528
Differential Revision: D9889990
Differential Version: 59236356
Differential Revision: D9889990
Differential Version: 59596351
Differential Revision: D9889990
Differential Version: 59619253
Differential Revision: D9889990
Differential Version: 59621759
Differential Revision: D10001033
Differential Version: 59624116
Differential Revision: D9889990
Differential Version: 59625764
Differential Revision: D10022853
Differential Version: 59663517
Differential Revision: D9889990
Differential Version: 59666455
Differential Revision: D10001033
Differential Version: 59666456
Differential Revision: D10022853
Differential Version: 59668026
Differential Revision: D10022853
Differential Version: 59730149
Differential Revision: D10022853
Differential Version: 59769948
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.

approved in fbcode

Differential Revision: D10022853
Differential Version: 59783324
Differential Revision: D10022853
Differential Version: 59800206
Differential Revision: D10022853
Differential Version: 59886111
Differential Revision: D10022853
Differential Version: 59892819
Differential Revision: D10069839
Differential Version: 59893026
Differential Revision: D10022853
Differential Version: 60013750
Differential Revision: D10069839
Differential Version: 60026094
Differential Revision: D10069839
Differential Version: 60055449
Differential Revision: D10069839
Differential Version: 60058845
Differential Revision: D10022853
Differential Version: 60136698
Differential Revision: D10069839
Differential Version: 60144811
Differential Revision: D10069839
Differential Version: 60192180
@jerryzh168 jerryzh168 changed the base branch from export-D10022853 to master October 10, 2018 00:55
Differential Revision: D10069839
Differential Version: 60213167
@jerryzh168
Copy link
Contributor Author

@pytorchbot retest this please

Differential Revision: D10069839
Differential Version: 60272376
zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 10, 2018
Summary:
Pull Request resolved: pytorch/pytorch#12304

- make ExtractDeviceOption to be a free function.
- Add a Strorage(at::Device) constructor in order to preserve the device_id.

Reviewed By: dzhulgakov

Differential Revision: D10069839

fbshipit-source-id: a5f3994a39bdf1b7503b39bb42c228e438b52bfa
@soumith soumith deleted the export-D10069839 branch February 21, 2019 12:11
@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.

2 participants