Skip to content

Fixed the quantize parameter will be failed if we just follow the Prepare Data & Run section #1216

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

Closed
wants to merge 3 commits into from

Conversation

shiqinwen
Copy link

@shiqinwen shiqinwen commented Apr 28, 2023

We need to pass in type 2, 3 rather than q4_0, q4_1 etc.
type = 2 - q4_0
type = 3 - q4_1

By given `q4_0` as the type will cause this error `llama_model_quantize: failed to quantize: invalid output file type 0`. 
And per the doc, the type should be 2 if we need q4_0
  type = 2 - q4_0
  type = 3 - q4_1

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…e-param

Correct the parameters of type given.
@sw
Copy link
Contributor

sw commented Apr 28, 2023

#1191 allows passing the type name, this change shouldn't be necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants