-
Notifications
You must be signed in to change notification settings - Fork 589
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
[HfFileSystem] Less /paths-info calls #2271
Conversation
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
This is also causing bugs in
could you take a look @Wauplin ? It would be cool to release this fix when you return :p |
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.
Conceptual approval. @lhoestq have you tested it with datasets
to check loading time is faster?
It's been in prod in datasets-viewer and it fixes the HfHubHTTPError (Too Many Requests) both for the FineWeb's viewer and also for loading the mmteb datasets in |
Great, thanks for confirming 👍 |
* less paths-info calls * fix tests
@lhoestq hot-fix released in 0.23.1: /~https://github.com/huggingface/huggingface_hub/releases/tag/v0.23.1 |
thank you !! |
...or we get rate-limited in the dataset viewer (e.g. for FineWeb)
cc @severo