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

Allow https://huggingface.co/ prefix #8

Closed
wants to merge 1 commit into from

Conversation

jonashaag
Copy link

As discussed in asteroid-team/asteroid#401 I'm submitting a patch to allow repo URLs as download URLs. Curious what you think of the change.

@julien-c
Copy link
Member

I'd be ok with merging this, but I feel like in terms of API it should probably be implemented in Asteroid (or another client library) if you need it, no?

I don't have a strong opinion though, so I'll leave this open and see if others want to chime in.

@mpariente
Copy link

Sorry, I'm late about that.
I think copy pasting the URL to have a model is pretty intuitive, but if you don't want to have that here, that's also fine, we can implement it in Asteroid.

What do we decide?

@julien-c
Copy link
Member

sorry about the delay here.

I'd rather not do it here and leave it to the library, but I'll export HUGGINGFACE_CO_REPO_URL_BASE so you don't have to hardcode anything on your side

@julien-c julien-c closed this Feb 10, 2021
julien-c pushed a commit that referenced this pull request May 21, 2021
julien-c pushed a commit that referenced this pull request May 21, 2021
Wauplin pushed a commit that referenced this pull request Nov 5, 2024
* Bare minimum download path

* Bare minimum download path

* Providing the sha for file upload

* Fixing the download path

* Fixing style and quality issues

* Conditionally choosing the XET upload path only if the user has hf_xet installed

* Remove the need to pass a sha256 into hf_xet (#8)

---------

Co-authored-by: Joseph Godlewski <[email protected]>
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