Skip to content

Removed torch.cholesky and added torch.linalg.cholesky#3274

Open
IvanYashchuk wants to merge 4 commits intopytorch:masterfrom
IvanYashchuk:remove-deprecated-cholesky
Open

Removed torch.cholesky and added torch.linalg.cholesky#3274
IvanYashchuk wants to merge 4 commits intopytorch:masterfrom
IvanYashchuk:remove-deprecated-cholesky

Conversation

@IvanYashchuk
Copy link
Copy Markdown
Contributor

@IvanYashchuk IvanYashchuk commented Jan 3, 2022

torch.cholesky was deprecated in the 1.9 release and will be removed in the next 1.11 release. This PR removes cholesky and replaces it with the linalg_cholesky.

pytorch/pytorch#70979 that removes deprecated torch.cholesky from PyTorch is currently blocked by pytorch/xla.

@IvanYashchuk
Copy link
Copy Markdown
Contributor Author

CI checks pass. @JackCaoG, could you please review this PR and #3273, #3272?

@JackCaoG
Copy link
Copy Markdown
Collaborator

@IvanYashchuk Will do today

@JackCaoG JackCaoG self-requested a review January 28, 2022 19:18
Copy link
Copy Markdown
Collaborator

@JackCaoG JackCaoG left a comment

Choose a reason for hiding this comment

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

Please give me a ping before you merge the pytorch pr

clang-format-7

Try branch with linalg_qr symbol

Revert "Try branch with linalg_qr symbol"

This reverts commit db3e9a7.

Update tensor_methods

Restore formatting

use tensor_methods::linalg_cholesky

Remove cholesky
@IvanYashchuk IvanYashchuk force-pushed the remove-deprecated-cholesky branch from f01f21e to 16d12d4 Compare April 5, 2023 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants