Skip to content

Back out "[pytorch][PR] Revert "Move CreateContext to global registry (#11688)"" #12121

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Sep 26, 2018

Stack:
    :black_circle:  #12121 Back out "[pytorch][PR] Revert "Move CreateContext to global registry (#11688)""  💚
    :white_circle:  #12029 [pt1][tensor] New in StaticContext returns at::DataPtr  💛

Original commit changeset: 6ca9de65b707

Differential Revision: D10033396

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 via ghapprove D10033396

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 via ghapprove D10033396

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 via ghapprove -s D10033396

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 via ghapprove -s D10033396

@jerryzh168 jerryzh168 changed the title Back out "[pytorch][PR] Revert "Move CreateContext to global registry (#11688)"" (#12055) Back out "[pytorch][PR] Revert "Move CreateContext to global registry (#11688)"" Sep 27, 2018
@jerryzh168
Copy link
Contributor Author

@pytorchbot retest this please

@jerryzh168
Copy link
Contributor Author

@pytorchbot retest this please

1 similar comment
@jerryzh168
Copy link
Contributor Author

@pytorchbot retest this please

…pytorch#11688)"" (pytorch#12121)

Summary:
Pull Request resolved: pytorch#12121

Pull Request resolved: pytorch#12055

Original commit changeset: 6ca9de65b707

Reviewed By: ezyang

Differential Revision: D10033396

fbshipit-source-id: 0936ccbeae529119586e41dd1a687f5ef1f85c8f
zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 1, 2018
… (#11688)"" (#12121)

Summary:
Pull Request resolved: pytorch/pytorch#12121

Pull Request resolved: pytorch/pytorch#12055

Original commit changeset: 6ca9de65b707

Reviewed By: ezyang

Differential Revision: D10033396

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