Skip to content

Move torch.cholesky_solve into torch.linalg. #61658

@lezcano

Description

@lezcano

This function is ready to be moved. It just needs

  • A review of its docs in and args to go in line with torch.linalg.
  • Make torch.cholesky_solve an alias.

The only point left to discuss is: do we want to name it torch.linalg.cho_solve in line with scipy or do we want to maintain the cholesky_solve name? In my opinion our name is better, but perhaps it's better to change it for compatibility purposes?

cc @jianyuh @nikitaved @pearu @mruberry @heitorschueroff @walterddr @IvanYashchuk @xwang233 @lezcano

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNot as big of a feature, but technically not a bug. Should be easy to fixmodule: linear algebraIssues related to specialized linear algebra operations in PyTorch; includes matrix multiply matmultriagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate module

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions