Skip to content

Fix skorch example: Replace unavailable OpenML MNIST#338

Merged
not522 merged 3 commits intooptuna:mainfrom
sotagg:fix/skorch-mnist
Nov 12, 2025
Merged

Fix skorch example: Replace unavailable OpenML MNIST#338
not522 merged 3 commits intooptuna:mainfrom
sotagg:fix/skorch-mnist

Conversation

@sotagg
Copy link
Copy Markdown
Contributor

@sotagg sotagg commented Nov 6, 2025

Motivation

This PR fixes the first error of #336 by replacing the unavailable OpenML MNIST dataset with torchvision.datasets.MNIST.

@not522 not522 self-assigned this Nov 12, 2025
Copy link
Copy Markdown
Member

@not522 not522 left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM!

@not522 not522 merged commit 55bbf69 into optuna:main Nov 12, 2025
5 checks passed
@not522 not522 added this to the v4.7.0 milestone Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants