-
Notifications
You must be signed in to change notification settings - Fork 237
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
Fix datasets download imports #143
Conversation
Thanks for the PR @albertvillanova. This is also fixes #142 |
Ah you beat me to it! 😄 (Edit: deleted my branch) I think there's another one to fix under from datasets.utils import DownloadConfig |
FYI I also opened huggingface/datasets#4497 to re-add them in Note that these imports are deprecated, so it's nice if you can use the new import :) |
These changes would require |
The documentation is not available anymore as the PR was closed or merged. |
@lvwerra with this fix, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this :)
CC: @pacman100