-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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
gguf-py: moved scripts directory #11116
Conversation
@ggerganov Pyright checks are failing in all pipelines because safetensors released version 0.5.1 to PyPi with a bug a few hours ago. Waiting on this fix to get merged and published. If there's too much delay, we can pin the package to 0.5.0. |
safetensor version 0.5.2 was just released. That fixed the issue. |
I think this commit broke the local detection of gguf library for non-installed setups should be instead:
This will need to be changed in 7 other files. |
@LostRuins I'm seeing several places where this needs to be updated. Will push the fix as soon as possible. Thanks for pointing that out! |
* Moved scripts dir and fixed pyproject.toml * updated readme * fixed README urls * bump pypi gguf to v0.14.0 * retrigger ci * empty commit - trigger ci
When installing the
gguf
pypi package, only one root level package should be registered. Currently two seprate packages get registered (scripts
andgguf
) which can cause issues when your project already has ascripts
folder.This PR moves the scripts directory inside the
gguf
package, making it a subpackage ofgguf
.Fixes #11089 and Fixes #9566