-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Latest fsspec==2023.10.0 issue with streaming datasets #6330
Comments
I also encountered a similar error below.
Code to reproduce the issue:
Dependencies:
|
@humpydonkey FWIW setting fsspec down to 2023.9.2 fixed the issue
|
got it, thanks @ZachNagengast |
Thanks for reporting and for the investigation, @ZachNagengast! 🤗 We are investigating the root cause of the issue. In the meantime, we are going to pin fsspec < 2023.10.0. |
Close huggingface#6330 `fsspec.implementations.LocalFilesystem.protocol` was changed from `str` "file" to `tuple[str,...]` ("file", "local") in `fsspec>=2023.10.0` This commit supports both styles.
Close huggingface#6330 `fsspec.implementations.LocalFilesystem.protocol` was changed from `str` "file" to `tuple[str,...]` ("file", "local") in `fsspec>=2023.10.0` This commit supports both styles.
Close #6330 `fsspec.implementations.LocalFilesystem.protocol` was changed from `str` "file" to `tuple[str,...]` ("file", "local") in `fsspec>=2023.10.0` This commit supports both styles.
Close #6330 `fsspec.implementations.LocalFilesystem.protocol` was changed from `str` "file" to `tuple[str,...]` ("file", "local") in `fsspec>=2023.10.0` This commit supports both styles.
Close #6330 `fsspec.implementations.LocalFilesystem.protocol` was changed from `str` "file" to `tuple[str,...]` ("file", "local") in `fsspec>=2023.10.0` This commit supports both styles.
Close #6330 `fsspec.implementations.LocalFilesystem.protocol` was changed from `str` "file" to `tuple[str,...]` ("file", "local") in `fsspec>=2023.10.0` This commit supports both styles.
Fix for "NotImplementedError: Loading a streaming dataset cached in a LocalFileSystem is not supported yet." huggingface/datasets#6330 (comment)
You can also update
It will also update |
This seems to work fine in 2.19.0. Hopefully it will not break again |
not working for 2.19.1 ! |
Describe the bug
Loading a streaming dataset with this version of fsspec fails with the following error:
NotImplementedError: Loading a streaming dataset cached in a LocalFileSystem is not supported yet.
I suspect the issue is with this PR
fsspec/filesystem_spec#1381
Steps to reproduce the bug
2023.10.0
load_dataset("laion/gpt4v-emotion-dataset", split="train", streaming=True)
Expected behavior
Should stream the dataset as normal.
Environment info
datasets@main
fsspec==2023.10.0
The text was updated successfully, but these errors were encountered: