New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[doc] Fix info on the shape of pivots in torch.lu
+ more info on what and how they encode permutations.
#46844
Conversation
7fb0756
to
f1d0849
Compare
torch.lu
+ more info on what and how they encode permutations.torch.lu
+ more info on what and how they encode permutations.
0d47a58
to
cce1089
Compare
torch/functional.py
Outdated
- **pivots** (*IntTensor*): the pivots of size :math:`(*, min(m, n))`. | ||
``pivots`` stores all the intermediate transpositions of rows. | ||
The final permutation ``perm`` could be reconstructed by | ||
applying :math:`swap(perm[i], perm[pivots[i] - 1])` for :math:`i = 0, \ldots, pivots.size(-1) - 1`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
swap
, perm
and pivots
could be in \text{}
, otherwise they'll be formatting like math.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replaced with
to replicate the code insertion.
cce1089
to
68e1f41
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks.
@pytorchbot merge this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ezyang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
As per title.