-
Notifications
You must be signed in to change notification settings - Fork 26.3k
[Easy][FSDP] Minor doc fixes #74214
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
[Easy][FSDP] Minor doc fixes #74214
Conversation
[ghstack-poisoned]
CI Flow Status⚛️ CI FlowRuleset - Version:
|
🔗 Helpful links
💊 CI failures summary and remediationsAs of commit bbfbae2 (more details on the Dr. CI page):
🕵️ 1 new failure recognized by patternsThe following CI failures do not appear to be due to upstream breakages:
|
| Job | Step | Action |
|---|---|---|
| Unknown | 🔁 rerun | |
| Test | 🔁 rerun |
This comment was automatically generated by Dr. CI (expand for details).
Please report bugs/suggestions to the (internal) Dr. CI Users group.
rohan-varma
left a comment
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.
Thank you for improving our documentation!
[ghstack-poisoned]
[ghstack-poisoned]
|
@awgu has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
ghstack-source-id: 4f33833 Pull Request resolved: pytorch#74214
Differential Revision: [D35045122](https://our.internmc.facebook.com/intern/diff/D35045122) [ghstack-poisoned]
|
@awgu has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
2 similar comments
|
@awgu has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
|
@awgu has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Differential Revision: [D35045122](https://our.internmc.facebook.com/intern/diff/D35045122) [ghstack-poisoned]
|
@awgu has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: Pull Request resolved: #74214 Test Plan: Imported from OSS Reviewed By: saketh-are Differential Revision: D35045122 Pulled By: awgu fbshipit-source-id: 701181cfdb93cb4ebf8e036a9f5c118f38005d6d
|
Hey @awgu. |
|
Reverting--broke docs job as can be seen in PR CI as well. Please make sure the docs job passes when relanding this PR. |
|
This pull request has been reverted by 47acdfa. To re-land this change, please open another pull request, assignthe same reviewers, fix the CI failures that caused the revert and make sure that the failing CI runs on the PR by applying the proper ciflow label (e.g., ciflow/trunk). |
Reland of #74214. Includes fixing #74694. Differential Revision: [D35188146](https://our.internmc.facebook.com/intern/diff/D35188146) [ghstack-poisoned]
Reland of #74214. Includes fixing #74694. Differential Revision: [D35188146](https://our.internmc.facebook.com/intern/diff/D35188146) [ghstack-poisoned]
Reland of #74214. Includes fixing #74694. Differential Revision: [D35188146](https://our.internmc.facebook.com/intern/diff/D35188146) [ghstack-poisoned]
Reland of #74214. Includes fixing #74694. Differential Revision: [D35188146](https://our.internmc.facebook.com/intern/diff/D35188146) [ghstack-poisoned]
Reland of #74214. Includes fixing #74694. Differential Revision: [D35188146](https://our.internmc.facebook.com/intern/diff/D35188146) [ghstack-poisoned]
Reland of #74214. Includes fixing #74694. Differential Revision: [D35188146](https://our.internmc.facebook.com/intern/diff/D35188146) [ghstack-poisoned]
|
This pull request has been reverted by 47acdfa. To re-land this change, please open another pull request, assignthe same reviewers, fix the CI failures that caused the revert and make sure that the failing CI runs on the PR by applying the proper ciflow label (e.g., ciflow/trunk). |
Stack from ghstack:
Differential Revision: D35045122