Skip to content
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

Xet Docs for huggingface_hub #2899

Open
wants to merge 9 commits into
base: xet-integration
Choose a base branch
from

Conversation

rajatarya
Copy link

Docs describing enabling and using Xet Storage with huggingface_hub.

These docs reference docs in HF Hub, which will be anchored here: https://huggingface.co/docs/hub/repositories (as a new Storage section).

The PR for those docs are here: huggingface/hub-docs#1622 (draft PR for now).

Copy link

@jsulz jsulz left a comment

Choose a reason for hiding this comment

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

Overall looks good to me. Added some comments, but none are blockers IMO.

Copy link
Member

@Pierrci Pierrci left a comment

Choose a reason for hiding this comment

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

nice

@julien-c
Copy link
Member

julien-c commented Mar 13, 2025

weird BTW, doc-builder did not deploy this PR's doc? cc @coyotte508 maybe

Nvm, it's here now: https://moon-ci-docs.huggingface.co/docs/huggingface_hub/pr_2899/en/index

@HuggingFaceDocBuilderDev

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.

@@ -541,3 +541,92 @@ Will free 8.6G
>>> delete_strategy.execute()
Cache deletion done. Saved 8.6G.
```

## Xet Cache
Copy link
Member

Choose a reason for hiding this comment

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

is this section at the right location in the page, @Wauplin?

rajatarya and others added 7 commits March 13, 2025 10:27
Co-authored-by: Pierric Cistac <Pierrci@users.noreply.github.com>
Co-authored-by: Pierric Cistac <Pierrci@users.noreply.github.com>
Co-authored-by: Pierric Cistac <Pierrci@users.noreply.github.com>
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.

5 participants