Skip to content

[wip] Add batched linear system of equations support to torch.gesv#4612

Closed
zou3519 wants to merge 5 commits intopytorch:masterfrom
zou3519:super-gesv
Closed

[wip] Add batched linear system of equations support to torch.gesv#4612
zou3519 wants to merge 5 commits intopytorch:masterfrom
zou3519:super-gesv

Conversation

@zou3519
Copy link
Copy Markdown
Contributor

@zou3519 zou3519 commented Jan 11, 2018

Fixes #3164

Picks up from #4502. I rewrote the code in ATen, which led to nice support for the following features:

  • can take in non-batched matrices as well as batched matrices
  • arbitrary batched dims
  • broadcasting of arbitrary batch dims

A side note:

  • torch.gesv() on tensors is different from torch.gesv on Variable. On tensor, the behavior falls back to THTensor_(gesv) because ATen doesn't support dispatching to tensors yet. When tensors/variables merge this should be fixed.

Test Plan

New unit tests to test arbitrary batched dimensions and broadcasting.

@pytorchbot
Copy link
Copy Markdown
Collaborator

@zou3519, thanks for your PR! We identified @zdevito to be a potential reviewer.

@zou3519 zou3519 force-pushed the super-gesv branch 2 times, most recently from db6ee32 to b2757fe Compare January 12, 2018 17:08
@zou3519 zou3519 changed the title Add batched linear system of equations support to torch.gesv [wip] Add batched linear system of equations support to torch.gesv Jan 29, 2018
@zou3519
Copy link
Copy Markdown
Contributor Author

zou3519 commented Mar 29, 2018

Closing in favor of a new one.

@zou3519 zou3519 closed this Mar 29, 2018
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.

[Feature Request] batched gesv

2 participants