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

Fix non-existing parameters in docstrings #7025

Merged
merged 1 commit into from
Dec 13, 2022
Merged

Conversation

kit1980
Copy link
Member

@kit1980 kit1980 commented Dec 8, 2022

No description provided.

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.

Thanks for the PR @kit1980. I left one comment below, let me know what you think.

torchvision/transforms/transforms.py Show resolved Hide resolved
@kit1980
Copy link
Member Author

kit1980 commented Dec 13, 2022

@datumbox I've updated the PR, please take another look.

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 @kit1980.

@datumbox datumbox merged commit 0dceac0 into main Dec 13, 2022
@github-actions
Copy link

Hey @datumbox!

You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

@datumbox datumbox deleted the sdym/docstring-args branch December 13, 2022 08:54
facebook-github-bot pushed a commit that referenced this pull request Dec 16, 2022
Reviewed By: YosuaMichael

Differential Revision: D42046595

fbshipit-source-id: e663b8e9fb45dbdc130f7b714d5808301e59ebe2
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.

3 participants