Skip to content

Turn some th_ prefixes into _th_ prefixes for conformity.#13128

Closed
gchanan wants to merge 1 commit intopytorch:masterfrom
gchanan:_th_prefix_minor
Closed

Turn some th_ prefixes into _th_ prefixes for conformity.#13128
gchanan wants to merge 1 commit intopytorch:masterfrom
gchanan:_th_prefix_minor

Conversation

@gchanan
Copy link
Copy Markdown
Contributor

@gchanan gchanan commented Oct 25, 2018

This is the same as #12889 with the addmm changes stripped out, since that appears to cause onnx broadcasting issues I don't understand.

Copy link
Copy Markdown
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 Oct 26, 2018
Summary:
This is the same as pytorch/pytorch#12889 with the addmm changes stripped out, since that appears to cause onnx broadcasting issues I don't understand.
Pull Request resolved: pytorch/pytorch#13128

Reviewed By: ezyang

Differential Revision: D10853911

Pulled By: gchanan

fbshipit-source-id: 08ec8629331972f0c332ccd036980fd9c87562b0
@ezyang ezyang added the merged label Jun 25, 2019
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
)

Summary:
This is the same as pytorch#12889 with the addmm changes stripped out, since that appears to cause onnx broadcasting issues I don't understand.
Pull Request resolved: pytorch#13128

Reviewed By: ezyang

Differential Revision: D10853911

Pulled By: gchanan

fbshipit-source-id: 08ec8629331972f0c332ccd036980fd9c87562b0
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