Skip to content
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] linalg.ldl_factor: render the Shape of tensor A #99777

Closed
wants to merge 1 commit into from

Conversation

isdanni
Copy link
Contributor

@isdanni isdanni commented Apr 22, 2023

Summary: Fix: #96864

Test Plan: Please see GitHub tests.

@pytorch-bot pytorch-bot bot added the release notes: linalg_frontend release notes category label Apr 22, 2023
@pytorch-bot
Copy link

pytorch-bot bot commented Apr 22, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/99777

Note: Links to docs will display an error until the docs builds have been completed.

✅ 1 Unrelated Failure

As of commit 90a1625:

UNSTABLE - The following job failed but was likely due to flakiness present on trunk and has been marked as unstable:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45204660

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45204660

Copy link
Collaborator

@lezcano lezcano left a comment

Choose a reason for hiding this comment

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

Please, see the format that's used in the rest of linalg. In particular, we avoid using :math: as it takes quite a long time to render. Use single backticks instead.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45204660

isdanni added a commit to isdanni/pytorch that referenced this pull request Apr 22, 2023
Summary: Pull Request resolved: pytorch#99777

Differential Revision: D45204660

fbshipit-source-id: 58006c20691e10733714b98a4b62e3c40482f46c
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45204660

isdanni added a commit to isdanni/pytorch that referenced this pull request Apr 22, 2023
Summary: Pull Request resolved: pytorch#99777

Differential Revision: D45204660

fbshipit-source-id: 74feb22215b595e1599d5fdfebfbf84a76acdaec
isdanni added a commit to isdanni/pytorch that referenced this pull request Apr 23, 2023
Summary: Pull Request resolved: pytorch#99777

Differential Revision: D45204660

fbshipit-source-id: 92789ab16ebea78ef0385db2020e4fd2a8bfda0c
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45204660

@@ -871,7 +871,8 @@
""" + r"""

Args:
A (Tensor): tensor of shape (*, n, n) where * is zero or more batch dimensions consisting of symmetric or Hermitian matrices.
A (Tensor): tensor of shape `(*, n, n)` where `*` is zero or more batch dimensions
consisting of symmetric or Hermitian matrices.
`(*, n, n)` where `*` is one or more batch dimensions.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Feel free to delete this line as well, as it's repeated. Same in the other function.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, this line has extra indentation. Same in the other one

isdanni added a commit to isdanni/pytorch that referenced this pull request Apr 24, 2023
Summary: Pull Request resolved: pytorch#99777

Differential Revision: D45204660

fbshipit-source-id: a6e842c166a9b8d95e5f1d777ddd27654a35869b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45204660

Copy link
Collaborator

@lezcano lezcano left a comment

Choose a reason for hiding this comment

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

Thank you!

@isdanni isdanni changed the title [doc] linalg.ldl_factor: render the Shape of tensor A [Doc] linalg.ldl_factor: render the Shape of tensor A Apr 28, 2023
@github-actions
Copy link

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

@github-actions github-actions bot added the Stale label Jun 27, 2023
@isdanni
Copy link
Contributor Author

isdanni commented Jun 27, 2023

@lezcano @IvanYashchuk any plans to merge this? :)

@lezcano
Copy link
Collaborator

lezcano commented Jun 28, 2023

You can merge it yourself writing @pytorchbot merge in a comment :)

@isdanni
Copy link
Contributor Author

isdanni commented Jun 28, 2023

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Jun 28, 2023
@isdanni
Copy link
Contributor Author

isdanni commented Jun 28, 2023

@pytorchbot merge -r

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Successfully rebased export-D45204660 onto refs/remotes/origin/viable/strict, please pull locally before adding more changes (for example, via git checkout export-D45204660 && git pull --rebase)

pytorchmergebot pushed a commit to isdanni/pytorch that referenced this pull request Jun 28, 2023
Summary: Pull Request resolved: pytorch#99777

Differential Revision: D45204660

fbshipit-source-id: a6e842c166a9b8d95e5f1d777ddd27654a35869b
@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: This PR has internal changes and must be landed via Phabricator

Details for Dev Infra team Raised by workflow job

@isdanni
Copy link
Contributor Author

isdanni commented Jun 28, 2023

@pytorchbot merge -r

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Tried to rebase and push PR #99777, but it was already up to date. Try rebasing against main by issuing:
@pytorchbot rebase -b main

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: 1 jobs have failed, first few of them are: Meta Internal-Only Changes Check

Details for Dev Infra team Raised by workflow job

@isdanni
Copy link
Contributor Author

isdanni commented Jun 28, 2023

@pytorchbot merge -f

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 28, 2023

❌ 🤖 pytorchbot command failed:

@pytorchbot merge: error: argument -f/--force: expected one argument

usage: @pytorchbot merge [-f MESSAGE | -i] [-ic] [-r [{viable/strict,main}]]

Try @pytorchbot --help for more info.

@isdanni
Copy link
Contributor Author

isdanni commented Jun 28, 2023

@pytorchbot merge -f "test failure unrelated"

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 28, 2023

You are not authorized to force merges to this repository. Please use the regular @pytorchmergebot merge command instead

Summary: Pull Request resolved: pytorch#99777

Differential Revision: D45204660

fbshipit-source-id: c797c01364df96b0b71073244527f45c8ddba3ea
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45204660

@lezcano
Copy link
Collaborator

lezcano commented Jun 28, 2023

Since you imported this, you should import the rebased branch onto phabricator as well, I believe.

@isdanni
Copy link
Contributor Author

isdanni commented Jun 28, 2023

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request fb-exported Merged release notes: linalg_frontend release notes category Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[doc] linalg.ldl_factor : Shape of tensor A is rendered incorrectly
4 participants