Skip to content

[docs] Usage example missing for torchvision.utils.draw_bounding_boxes #3449

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

Closed
vadimkantorov opened this issue Feb 24, 2021 · 2 comments · Fixed by #3594
Closed

[docs] Usage example missing for torchvision.utils.draw_bounding_boxes #3449

vadimkantorov opened this issue Feb 24, 2021 · 2 comments · Fixed by #3594

Comments

@vadimkantorov
Copy link

https://pytorch.org/vision/master/utils.html?highlight=draw_bounding_boxes#torchvision.utils.draw_bounding_boxes

Return type is not described.

Also, it says: If filled, Resulting Tensor should be saved as PNG image.. What does word filled refer to? Where would it be saved? To some file path? Or encoded into the output tensor?

@vadimkantorov vadimkantorov changed the title [docs] Usage example missing for draw_bounding_boxes [docs] Usage example missing for torchvision.utils.draw_bounding_boxes Feb 24, 2021
@oke-aditya
Copy link
Contributor

oke-aditya commented Feb 25, 2021

Nice points @vadimkantorov . We recently added these utils and they are bit on the rough edge.
We have these TODO points in #3364 . I wanted to add this PR after #3330 .

@datumbox Since it will take time to add segmentation mask utility. Should I quickly patch #3364 ?

@datumbox
Copy link
Contributor

@oke-aditya Apologies for the delay. I'm currently OOO...

@fmassa Thoughts on how we could unblock Oke on his open PR or how to proceed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants