-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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
readme : update gpt4all instructions #980
Conversation
I am running into one issue with the bin:
|
As the error says, you don't have the |
Also, based off the model and message, you probably should be using the migrate script, not the convert script. That model file is already ggml, it just needs to be migrated to current version. |
there is no migrate script anymore, just one complex one-does-it-all convert script (#545) |
There's not a |
No. |
Welp. Good to know. Unfortunate that it has the same exact name as a previous |
Anyways this PR is just for the readme. I have another error, will share in
20 min.
…On Fri, Apr 14, 2023 at 12:32 MillionthOdin16 ***@***.***> wrote:
Welp. Good to know. Unfortunate that it has the same exact name as a
previous convert.py script that does a subset of tasks lol
—
Reply to this email directly, view it on GitHub
<#980 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAOZDW2DGXIYDFQYDV4WFUDXBGQ4HANCNFSM6AAAAAAW6Z2L5Y>
.
You are receiving this because you commented.Message ID: <ggerganov/llama.
***@***.***>
|
Exactly, this PR is about fixing readme. Any subsequent errors are out of scope of this PR and should be fixed by future PRs. |
Since this section is about Gpt4all, it might be helpful to enumerate the required files |
Thanks for cleaning up my oversight. |
Converting to draft since it seems that gpt4all conversion is broken, see #984 |
6b1d165
to
de129f7
Compare
Fixes #1036 |
suggest adding the SHASUMS so people can validate their conversions |
Co-authored-by: LostRuins Concedo <39025047+LostRuins@users.noreply.github.com>
fixes #975