Skip to content
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

Making root parameter optional for Vision Dataset #8124

Merged
merged 3 commits into from
Nov 20, 2023

Conversation

zshn25
Copy link
Contributor

@zshn25 zshn25 commented Nov 20, 2023

This resolves #8123 while maintaining backward compatibility

cc @vfdev-5

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
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 @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
60 of 64 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VisionDataset abstract class forces to set 'root' parameter, even if it is unused
4 participants