Skip to content

Add missing doc for get_video_backend #3643

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

Merged
merged 6 commits into from
Apr 7, 2021

Conversation

bjuncek
Copy link
Contributor

@bjuncek bjuncek commented Apr 7, 2021

addressing #3633

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Thanks @bjuncek , it looks like your editor automatically formatted some unrelated code: it's better to avoid this as it tends to break git blame. In big PRs, it can also complicate the review with noisy changes

bjuncek and others added 2 commits April 7, 2021 14:03
This reverts commit f9e847e
7d17834bece36614ca241c6db970.
@bjuncek
Copy link
Contributor Author

bjuncek commented Apr 7, 2021

Hi @NicolasHug - yeah makes sense. Sorry for the bother. I've reverted the change and only updated the docstring per your suggestion.
Please let me know if it's fine now and I'll ping someone for the merge.

Best,
Bruno

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Thanks! The docs CI was passing so I'll merge ;)

@NicolasHug NicolasHug changed the title Addressing #3633: adding missing doc Add missing doc for get_video_backend Apr 7, 2021
@NicolasHug NicolasHug merged commit e79a74e into pytorch:master Apr 7, 2021
facebook-github-bot pushed a commit that referenced this pull request Apr 13, 2021
Summary:

Reviewed By: NicolasHug

Differential Revision: D27706956

fbshipit-source-id: c5e3f4030b9df7081d72ea9a2e307cadb9f0a676

Co-authored-by: Nicolas Hug <[email protected]>
Co-authored-by: Nicolas Hug <[email protected]>
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