Skip to content

Conversation

@jambayk
Copy link
Contributor

@jambayk jambayk commented Jan 8, 2026

add the hf_xet package to avoid the warning when downloading models from HuggingFace.

I checked the repo and would need some opinion of developers where to put the fix: there are 2 places:

requirements.txt contains the base dependencies that are always installed olive/olive_config.json contains extra_dependencies for optional features that users can install as needed (like pip install olive-ai[gpu], pip install olive-ai[lora], etc.) Since the warning occurs during the getting started example (a basic use case), and hf_xet improves download performance from HuggingFace for models with Xet Storage, it makes sense to add it as a base dependency in requirements.txt.

Proposed change:
Add hf_xet to the requirements.txt

Describe your changes

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

add the hf_xet package to avoid the warning when downloading models from HuggingFace.

I checked the repo and would need some opinion of developers where to put the fix:
there are 2 places:

requirements.txt contains the base dependencies that are always installed
olive/olive_config.json contains extra_dependencies for optional features that users can install as needed (like pip install olive-ai[gpu], pip install olive-ai[lora], etc.)
Since the warning occurs during the getting started example (a basic use case), and hf_xet improves download performance from HuggingFace for models with Xet Storage, it makes sense to add it as a base dependency in requirements.txt.

Proposed change:
Add hf_xet to the requirements.txt
@jambayk jambayk closed this Jan 8, 2026
@jambayk jambayk deleted the jambayk/hf-xet branch January 8, 2026 15:28
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.

3 participants