-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Failure with source installations #3031
Comments
Thanks for the report! Was it working with a previous version of |
A different commit of the repository being used. Was able to pin-point: uv pip install -U peft@git+https://github.com/huggingface/peft.git@b0f1bb468ce58a26c1403db37e2ca8a76977b4c1 This works. Things break after: huggingface/peft#1326. |
Thanks! I'll look into this |
Interesting I can't reproduce with a simple project
|
This seems like something legitimately wrong with their submodule? |
It works with regular pip, though. |
I'm getting the same problem:
This works fine with normal pip. |
I'll take a look. |
@NeilGirdhar - I think you have a typo, three o's in Google. |
Ok so I've confirmed that pip doesn't recursively clone the contents of |
@charliermarsh I'm not sure what behavior is right here. I think we need to understand precisely what's wrong with that submodule setup to know if we should fail because of it or not. Otherwise... you could install from git and be silently missing a meaningful part of the package. |
Yeah I can just fix the repo itself if necessary. |
I'll try now. |
I think |
I am on my Apple M2 and uv-0.1.31.
Running:
uv pip install -U peft@git+https://github.com/huggingface/peft.git
.It's failing with:
Didn't happen last week.
FWIW,
uv pip install -U transformers@git+https://github.com/huggingface/transformers.git
doesn't fail.The text was updated successfully, but these errors were encountered: