Skip to content

Do not always set broadcast argument when exporting new onnx add and sub to caffe2#9597

Closed
bddppq wants to merge 1 commit intopytorch:masterfrom
bddppq:no-always-broadcast
Closed

Do not always set broadcast argument when exporting new onnx add and sub to caffe2#9597
bddppq wants to merge 1 commit intopytorch:masterfrom
bddppq:no-always-broadcast

Conversation

@bddppq
Copy link
Contributor

@bddppq bddppq commented Jul 19, 2018

No description provided.

Copy link
Member

@colesbury colesbury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This fixes the issue I was seeing with expressions like 1 - tensor

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.

@colesbury has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 30, 2018
…sub to caffe2

Summary: Pull Request resolved: pytorch#9597

Reviewed By: colesbury

Differential Revision: D8920575

Pulled By: bddppq

fbshipit-source-id: 97423e1bf6a20559d466d2ac56c9e74e10bfc129
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
…sub to caffe2

Summary: Pull Request resolved: pytorch#9597

Reviewed By: colesbury

Differential Revision: D8920575

Pulled By: bddppq

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