Skip to content

Conversation

@zshn25
Copy link
Contributor

@zshn25 zshn25 commented Nov 20, 2023

This resolves #8123 while maintaining backward compatibility

cc @vfdev-5

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 20, 2023

🔗 Helpful Links

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

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

✅ You can merge normally! (4 Unrelated Failures)

As of commit c686969 with merge base c7e2947 (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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

Copy link
Contributor

@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 @zshn25!

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 @zshn25 !

@NicolasHug NicolasHug merged commit 127e8a7 into pytorch:main Nov 20, 2023
@github-actions
Copy link

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

@zshn25 zshn25 deleted the patch-VisionDataset-root=None branch November 21, 2023 08:33
facebook-github-bot pushed a commit that referenced this pull request Jan 4, 2024
Reviewed By: vmoens Differential Revision: D52539001 fbshipit-source-id: b628a839f0f4477af09b3b4a9ce22951be3c7152 Co-authored-by: Nicolas Hug <contact@nicolas-hug.com> Co-authored-by: Nicolas Hug <nh.nicolas.hug@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment