Skip to content

dev(narugo): allow explicitly disable hf_transfer#2979

Merged
hanouticelina merged 2 commits intohuggingface:mainfrom
narugo1992:dev/disable_hftransfer
Apr 9, 2025
Merged

dev(narugo): allow explicitly disable hf_transfer#2979
hanouticelina merged 2 commits intohuggingface:mainfrom
narugo1992:dev/disable_hftransfer

Conversation

@narugo1992
Copy link
Copy Markdown
Contributor

for issue #2978

@hanouticelina
Copy link
Copy Markdown
Contributor

See my answer in the issue: #2978 (comment).

@narugo1992 narugo1992 force-pushed the dev/disable_hftransfer branch from 728b85c to b27dcf1 Compare April 7, 2025 12:18
@narugo1992
Copy link
Copy Markdown
Contributor Author

@hanouticelina updated

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

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.

Copy link
Copy Markdown
Contributor

@hanouticelina hanouticelina left a comment

Choose a reason for hiding this comment

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

looks good! it's a nice to have check given that hf_transfer does not leverage the range header

Copy link
Copy Markdown
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

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

Thanks!

@hanouticelina hanouticelina merged commit 8d675d5 into huggingface:main Apr 9, 2025
19 checks passed
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.

4 participants