Skip to content

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented Jul 9, 2025

This PR adds more docs for the rotated bounding boxes, as documents them as BETA, along with the new KeyPoints.
We'll also add a note about those things being BETA in our release notes.

You can view the built docs by clicking on the link from the pytorch-bot comment just below

Copy link

pytorch-bot bot commented Jul 9, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/vision/9144

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 55513b3 with merge base c50f694 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

# perspective is transformed with :class:`~torchvision.transforms.RandomPerspective`.
perspective_transformer = v2.RandomPerspective(distortion_scale=0.6, p=1.0)
perspective_imgs = [perspective_transformer(orig_img, orig_box) for _ in range(4)]
plot([(orig_img, orig_box)] + perspective_imgs, bbox_width=10)
Copy link
Member Author

Choose a reason for hiding this comment

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

@scotts @AntoineSimoulin , as discussed offline with both of you separately, the results of perspective were a bit fishy, so I removed it from this tutorial. We should try to investigate what's going on and potentially provide a bug fix, if we think this is a bug.

I ralso emoved Elastic below, not because it was buggy, but because it didn't seem to transform the original image in a relevant way for the purpose of this tutorial.

Copy link
Member

@AntoineSimoulin AntoineSimoulin left a comment

Choose a reason for hiding this comment

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

LGTM!

# -------
# Rotated bounding boxes also maintain their properties when the image is padded using
# :class:`~torchvision.transforms.Pad`.
# Using # :class:`~torchvision.transforms.Pad`:
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a leftover # here.

@NicolasHug NicolasHug merged commit b818d32 into pytorch:main Jul 9, 2025
58 checks passed
Copy link

github-actions bot commented Jul 9, 2025

Hey @NicolasHug!

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

AntoineSimoulin pushed a commit to AntoineSimoulin/vision that referenced this pull request Jul 9, 2025
facebook-github-bot pushed a commit that referenced this pull request Jul 31, 2025
Reviewed By: AntoineSimoulin

Differential Revision: D79175018

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

4 participants