-
Notifications
You must be signed in to change notification settings - Fork 8
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
add the handle to the model during loadmodel #27
Comments
It's already there, use |
right! I forgot I was already using it. sorry about that. and thanks for fixing that last issue so quickly. been working like charm now. btw, I now get these warning messages from llama.cpp. everything appears to be working. but I guess I am not in the loop to keep up with what this is supposed to be telling me.
|
also get this...
|
I think these are just warnings and they may have been added recently but I'm not entirely sure. |
I been using the meta parameters as an integral part of my setup. but I now realize there is no standardization on the meaning of the values. I cannot even rely on the model name in those parameters. They "could" be quite valuable to hint to my setup what a model can (and cannot do). Some even list the various merged models that the model was built from. Who has influence on what the key values are named? |
The values are set when the model is converted (i.e. they are coming from the source model). During the conversion to GGUF you can also override some of the values using the conversion script. And there are some values that can be set using |
Please add this to your loadmodel routine
So that I have access to the actual loaded model for running additional functions (such as load meta vars...)
The text was updated successfully, but these errors were encountered: