Skip to content

Conversation

@ggoossen
Copy link
Contributor

Summary:
addmm: Fix handling of case with empty tensor.
Currently these cause an error

Recreation of D18085389 without stacked diffs

Test Plan: test included

Differential Revision: D18122004

Summary:
addmm: Fix handling of case with empty tensor.
Currently these cause an error

Recreation of D18085389 without stacked diffs

Test Plan: test included

Differential Revision: D18122004

fbshipit-source-id: 74f1ee40d64a365205ed5b4f8a421a329c88c3a6
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D18122004

@zasdfgbnm
Copy link
Collaborator

@ggoossen Are you fixing #27619 ?

@ggoossen
Copy link
Contributor Author

@zasdfgbnm yes this fixes #27619

Copy link
Contributor

@soumith soumith left a comment

Choose a reason for hiding this comment

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

lgtm!

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in c60dee2.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 28, 2019
Summary:
Pull Request resolved: pytorch/pytorch#28613

addmm: Fix handling of case with empty tensor.
Currently these cause an error

Recreation of D18085389 without stacked diffs

Test Plan: test included

Differential Revision: D18122004

fbshipit-source-id: 71513c02ace691902553bea5ce9dc2538cca4c99
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants