KEMBAR78
[package] doc string cleanup extravaganza by Lilyjjo · Pull Request #59843 · pytorch/pytorch · GitHub
Skip to content

[package] doc string cleanup extravaganza #59843

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

[package] doc string cleanup extravaganza #59843

wants to merge 2 commits into from

Conversation

@Lilyjjo
Copy link
Contributor

@Lilyjjo Lilyjjo commented Jun 11, 2021 •
edited
Loading

Stack from ghstack:

Differential Revision: D29049342

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jun 11, 2021 •
edited
Loading

💊 CI failures summary and remediations

As of commit a0f0863 (more details on the Dr. CI page):

  • 3/3 failures introduced in this PR

3 failures not recognized by patterns:

Job Step Action
CircleCI pytorch_linux_xenial_cuda11_1_cudnn8_py3_gcc7_test1 Run tests 🔁 rerun
CircleCI pytorch_linux_xenial_cuda11_1_cudnn8_py3_gcc7_test2 Run tests 🔁 rerun
GitHub Actions Linux CI (pytorch-linux-xenial-py3.6-gcc5.4) / pytorch_python_doc_build Build Python Doc in Docker 🔁 rerun
This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@Lilyjjo
Copy link
Contributor Author

Lilyjjo commented Jun 11, 2021

@Lilyjjo has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@Lilyjjo Lilyjjo requested review from SplitInfinity and suo June 11, 2021 00:14
@suo
Copy link
Member

suo commented Jun 11, 2021

I'll let @SplitInfinity review this one as he mentioned he had some RST changes as well.

@suo suo removed their request for review June 11, 2021 00:20
Lilyjjo added a commit that referenced this pull request Jun 11, 2021
@Lilyjjo
[package] doc string cleanup extravaganza
f52d3b7 
ghstack-source-id: 810f32c
Pull Request resolved: #59843
@Lilyjjo
Copy link
Contributor Author

Lilyjjo commented Jun 11, 2021

@Lilyjjo has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@Lilyjjo merged this pull request in c7890b4.

@SplitInfinity SplitInfinity mentioned this pull request Jun 11, 2021
Closed
@facebook-github-bot facebook-github-bot deleted the gh/Lilyjjo/89/head branch June 14, 2021 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@SplitInfinity SplitInfinity SplitInfinity approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla signed Merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Lilyjjo @facebook-github-bot @suo @SplitInfinity