Skip to content

Fix: fix wrong error message in CelebA datasets when invalid split argument is passed in.#8866

Merged
NicolasHug merged 3 commits intopytorch:mainfrom
GdoongMathew:fix/celeb_a_split
Feb 19, 2025
Merged

Fix: fix wrong error message in CelebA datasets when invalid split argument is passed in.#8866
NicolasHug merged 3 commits intopytorch:mainfrom
GdoongMathew:fix/celeb_a_split

Conversation

@GdoongMathew
Copy link
Copy Markdown
Contributor

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Jan 16, 2025

🔗 Helpful Links

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

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

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

@GdoongMathew
Copy link
Copy Markdown
Contributor Author

A friendly ping to @NicolasHug for a review~

Copy link
Copy Markdown
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 a lot for the PR and the great tests @GdoongMathew

Copy link
Copy Markdown
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 a lot for the PR and the great tests @GdoongMathew

@NicolasHug NicolasHug merged commit 8bed9d8 into pytorch:main Feb 19, 2025
1 check passed
@GdoongMathew GdoongMathew deleted the fix/celeb_a_split branch February 19, 2025 16:34
facebook-github-bot pushed a commit that referenced this pull request Jul 11, 2025
…id split argument is passed in. (#8866)

Summary: Co-authored-by: Nicolas Hug <nh.nicolas.hug@gmail.com>

Reviewed By: scotts

Differential Revision: D77997063

fbshipit-source-id: 2a92b7c375ac0415d58dc6cade9d9b5f329e5c01
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.

Setting 3.14 or ["train"] to split argument of CelebA() gets the indirect error message

3 participants