-
Notifications
You must be signed in to change notification settings - Fork 308
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: add CLI #1221
gguf: add CLI #1221
Conversation
What will be the "official" command once it's released? UPDATE: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very nice, thanks
I deployed a test version of the package in my namespace too:
User can even install the package as global via |
I changed the binary name to Small note @mishig25 , we briefly discussed about |
perfectly fine IMO, and btw we might already have an implementation for this somewhere (i'm pretty sure i did some in the past) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
allowLocalFile: true, | ||
}); | ||
|
||
// TODO: print info about endianess |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we still need this todo ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, would be nice if we can make it outputs the same as gguf-py
script
Ref discussion: https://huggingface.slack.com/archives/C02CLHA19TL/p1740399079674399?thread_ts=1739968558.574099&cid=C02CLHA19TL
I'm trying with this command:
Output:
For reference, here is the output of
gguf_dump.py
: