Skip to content

Conversation

@weiyangfb
Copy link
Contributor

@weiyangfb weiyangfb commented Oct 30, 2018

@weiyangfb weiyangfb changed the title [wip] add backward for sparse_addmm [wip] add backward for sparse.addmm(D, S, D, alpha, beta) -> D Nov 1, 2018

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@weiyangfb weiyangfb changed the title [wip] add backward for sparse.addmm(D, S, D, alpha, beta) -> D [wip / sparse] backward for sparse.addmm(D, S, D, alpha, beta) -> D Nov 13, 2018
- exposed sparse.addmm
@weiyangfb weiyangfb changed the title [wip / sparse] backward for sparse.addmm(D, S, D, alpha, beta) -> D [sparse] backward for sparse.addmm(D, S, D, alpha, beta) -> D Nov 16, 2018
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.

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

@weiyangfb
Copy link
Contributor Author

cc @soumith , this is ready for review

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.

@weiyangfb is landing 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 Nov 27, 2018
Summary:
- introduce `sparse.addmm()` with backward for sparse matrix input for pytorch/pytorch#12308
Pull Request resolved: pytorch/pytorch#13345

Differential Revision: D13094070

Pulled By: weiyangfb

fbshipit-source-id: 136c08c3ca9bafb20577b60dd43d31c3e5cd5461
@ezyang ezyang added the merged label Jun 25, 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