Skip to content

Deprecate torch.lu_solve#77637

Closed
lezcano wants to merge 10 commits intogh/Lezcano/75/basefrom
gh/Lezcano/75/head
Closed

Deprecate torch.lu_solve#77637
lezcano wants to merge 10 commits intogh/Lezcano/75/basefrom
gh/Lezcano/75/head

Conversation

@lezcano
Copy link
Copy Markdown
Collaborator

@lezcano lezcano commented May 17, 2022

Stack from ghstack:

Relanding #73806

BC-breaking note

Deprecate torch.lu_solve in favor of linalg.lu_solve

The new operation has a notation consistent with linalg.solve, and has an extra parameter adjoint=False. The update rule is as follows:

Version 1.12.0

X = torch.lu_solve(B, LU, pivots)

Version 1.13.0

X = linalg.lu_solve(LU, pivots, B)

**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

[ghstack-poisoned]
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented May 17, 2022

🔗 Helpful links

✅ No Failures (0 Pending)

As of commit 8641d3e (more details on the Dr. CI page):

Expand to see more

💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@lezcano lezcano added module: linear algebra Issues related to specialized linear algebra operations in PyTorch; includes matrix multiply matmul release notes: linalg_frontend release notes category topic: deprecation topic category labels May 17, 2022
@lezcano lezcano requested review from malfet and removed request for IvanYashchuk and nikitaved May 17, 2022 09:08
lezcano added 9 commits May 17, 2022 10:31
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Relanding #73806

[ghstack-poisoned]
@lezcano
Copy link
Copy Markdown
Collaborator Author

lezcano commented Jun 7, 2022

@pytorchbot merge

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

@pytorchbot successfully started a merge job. Check the current status here

facebook-github-bot pushed a commit that referenced this pull request Jun 8, 2022
Summary:
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Pull Request resolved: #77637

Approved by: https://github.com/malfet

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/c6215b343cec6a820890e5d8c1aaf3253eb3cf1a

Reviewed By: osalpekar

Differential Revision: D36993330

Pulled By: osalpekar

fbshipit-source-id: 341d2249c26f806d318f6f4fe34d5bd851f6436d
@facebook-github-bot facebook-github-bot deleted the gh/Lezcano/75/head branch June 11, 2022 14:17
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 25, 2026
**BC-breaking note**:

This PR deprecates `torch.lu_solve` in favor of `torch.linalg.lu_solve_factor`.
A upgrade guide is added to the documentation for `torch.lu_solve`.

Note this PR DOES NOT remove `torch.lu_solve`.

Pull Request resolved: pytorch#77637

Approved by: https://github.com/malfet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged module: linear algebra Issues related to specialized linear algebra operations in PyTorch; includes matrix multiply matmul open source release notes: linalg_frontend release notes category topic: deprecation topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants