Skip to content

Conversation

lezwon
Copy link
Contributor

@lezwon lezwon commented Apr 7, 2022

Fixes #5725

@lezwon lezwon marked this pull request as draft April 7, 2022 13:23
@lezwon lezwon marked this pull request as ready for review April 7, 2022 14:16
Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

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

Two minor comments. Otherwise LGTM.

Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

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

Thanks @lezwon, LGTM!

@pmeier pmeier requested a review from NicolasHug April 19, 2022 06:42
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!

@pmeier pmeier merged commit 669d565 into pytorch:main Apr 19, 2022
@github-actions
Copy link

Hey @pmeier!

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

@lezwon lezwon deleted the 5725-font-warning branch April 20, 2022 10:37
facebook-github-bot pushed a commit that referenced this pull request May 5, 2022
Summary:
* add warning if font is not set

* added quotes

* added changes after review

* Apply suggestions from code review

* escaped warning string

* Apply suggestions from code review

* updated warning message in test

* refactored code

* Apply suggestions from code review

Reviewed By: jdsgomes, NicolasHug

Differential Revision: D36095644

fbshipit-source-id: efc171d4f4a5567b48b9365dc35e7d4ac45922a5

Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Co-authored-by: Vasilis Vryniotis <datumbox@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.

torchvision.utils.draw_bounding_boxes set font_size
6 participants