Skip to content

Conversation

@gchanan
Copy link
Contributor

@gchanan gchanan commented Aug 6, 2018

Turns out some people are using this via the C-API.

Turns out some people are using this via the C-API.
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.

@gchanan
Copy link
Contributor Author

gchanan commented Aug 6, 2018

CC @alsrgv.

@ezyang
Copy link
Contributor

ezyang commented Aug 6, 2018

I do wonder how we're going to prevent people from using these functions, but I guess we can cross that bridge when we get there.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Aug 6, 2018
Summary:
Turns out some people are using this via the C-API.
Pull Request resolved: pytorch/pytorch#10259

Differential Revision: D9180135

Pulled By: gchanan

fbshipit-source-id: 68f59beabf7f8093e67581d7e7ebfe8dff9e6b69
@alsrgv
Copy link

alsrgv commented Aug 8, 2018

Thanks for adding it back!

goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
Turns out some people are using this via the C-API.
Pull Request resolved: pytorch#10259

Differential Revision: D9180135

Pulled By: gchanan

fbshipit-source-id: 68f59beabf7f8093e67581d7e7ebfe8dff9e6b69
@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.

4 participants