-
Notifications
You must be signed in to change notification settings - Fork 12.4k
common: add config presets for falcon #14638
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
base: master
Are you sure you want to change the base?
Conversation
common/arg.cpp
Outdated
{"--fim-falcon-7b-spec"}, | ||
string_format("use quantized Falcon 7B model (note: can download weights from the internet)"), | ||
[](common_params & params) { | ||
params.model.hf_repo = "maddes8cht/tiiuae-falcon-7b-gguf"; |
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.
Presets should only use models hosted on ggml-org
.
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.
ahh ok got it. I see that falcon is not in ggml-org, I will add another model then
common/arg.cpp
Outdated
@@ -3423,5 +3423,21 @@ common_params_context common_params_parser_init(common_params & params, llama_ex | |||
} | |||
).set_examples({LLAMA_EXAMPLE_SERVER})); | |||
|
|||
add_opt(common_arg( | |||
{"--fim-falcon-7b-spec"}, |
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.
This identifier is not correct. It indicates:
- A Fill-In-the-Middle (FIM) model
- Speculative-decoding enabled (i.e. target + draft model)
You can call it --chat-falcon-7b
common/arg.cpp
Outdated
string_format("use quantized Falcon 7B model (note: can download weights from the internet)"), | ||
[](common_params & params) { | ||
params.model.hf_repo = "maddes8cht/tiiuae-falcon-7b-gguf"; | ||
params.model.hf_file = "tiiuae-falcon-7b-Q6_K.gguf"; |
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.
We use Q8_0
format in the presets.
added config presets for falcon -7b -instruct (using Q6_K)
towards #10932