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

Show PILToTensor in docs #4546

Merged
merged 3 commits into from
Oct 7, 2021
Merged

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented Oct 6, 2021

Looks like this was missing

Fixes #4225

cc @vfdev-5 @datumbox

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Oct 6, 2021

@NicolasHug there was an issue for that and someone wanted to work on that as well.

#4225

@NicolasHug
Copy link
Member Author

Thanks for the ref @vfdev-5
@harishsdev I cannot find an open PR or yours addressing the original issue. Do you mind if we merge this one?

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

We should get a patch in by tomorrow to meet the deadline for cherrypicking it into the release branch.

@NicolasHug NicolasHug merged commit 9337873 into pytorch:main Oct 7, 2021
@NicolasHug
Copy link
Member Author

As @datumbox said it would be a shame to not get this into the release, so I'll merge now. Thanks for the reviews!

NicolasHug added a commit to NicolasHug/vision that referenced this pull request Oct 7, 2021
* Add docs for PILToTensor

* Update docs/source/transforms.rst

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>
NicolasHug added a commit that referenced this pull request Oct 7, 2021
* Add docs for PILToTensor

* Update docs/source/transforms.rst

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>
facebook-github-bot pushed a commit that referenced this pull request Oct 8, 2021
Summary:
* Add docs for PILToTensor

* Update docs/source/transforms.rst

Reviewed By: NicolasHug

Differential Revision: D31505564

fbshipit-source-id: 9d8d4cf1097d3fef19e3c542aee799d684717020

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>
Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>
mszhanyi pushed a commit to mszhanyi/vision that referenced this pull request Oct 19, 2021
* Add docs for PILToTensor

* Update docs/source/transforms.rst

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>
cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
* Add docs for PILToTensor

* Update docs/source/transforms.rst

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>

Co-authored-by: Aditya Oke <47158509+oke-aditya@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing documentation for PILToTensor transform
5 participants