Skip to content
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

Merged
merged 1 commit into from
Apr 23, 2023
Merged

readme : update gpt4all instructions #980

merged 1 commit into from
Apr 23, 2023

Conversation

prusnak
Copy link
Collaborator

@prusnak prusnak commented Apr 14, 2023

fixes #975

@robbintt
Copy link

robbintt commented Apr 14, 2023

I am running into one issue with the bin:

python convert.py models/gpt4all-lora-quantized.bin
Loading model file models/gpt4all-lora-quantized.bin
Traceback (most recent call last):
  File "convert.py", line 1145, in <module>
    main()
  File "convert.py", line 1133, in main
    vocab = load_vocab(vocab_dir)
  File "convert.py", line 1075, in load_vocab
    raise FileNotFoundError(f"Could not find tokenizer.model in {path} or its parent; if it's in another directory, pass the directory as --vocab-dir")
FileNotFoundError: Could not find tokenizer.model in models or its parent; if it's in another directory, pass the directory as --vocab-dir```

@prusnak
Copy link
Collaborator Author

prusnak commented Apr 14, 2023

As the error says, you don't have the models/tokenizer.model file. You need to obtain it from the same place you are obtaining your LLaMa weights.

@MillionthOdin16
Copy link

I am running into one issue with the bin:

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.

@prusnak
Copy link
Collaborator Author

prusnak commented Apr 14, 2023

migrate script, not the convert script

there is no migrate script anymore, just one complex one-does-it-all convert script (#545)

@MillionthOdin16
Copy link

There's not a migrate-ggml-2023-03-30-pr613.py script?

@prusnak
Copy link
Collaborator Author

prusnak commented Apr 14, 2023

There's not a migrate-ggml-2023-03-30-pr613.py script?

No.

@MillionthOdin16
Copy link

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

@robbintt
Copy link

robbintt commented Apr 14, 2023 via email

@prusnak
Copy link
Collaborator Author

prusnak commented Apr 14, 2023

Anyways this PR is just for the readme. I have another error, will share in
20 min.

Exactly, this PR is about fixing readme. Any subsequent errors are out of scope of this PR and should be fixed by future PRs.

@robbintt
Copy link

Since this section is about Gpt4all, it might be helpful to enumerate the required files

@comex
Copy link
Contributor

comex commented Apr 15, 2023

Thanks for cleaning up my oversight.

@prusnak prusnak marked this pull request as draft April 15, 2023 08:27
@prusnak
Copy link
Collaborator Author

prusnak commented Apr 15, 2023

Converting to draft since it seems that gpt4all conversion is broken, see #984

@prusnak prusnak mentioned this pull request Apr 15, 2023
@prusnak prusnak force-pushed the readme-convert branch 2 times, most recently from 6b1d165 to de129f7 Compare April 17, 2023 17:57
@prusnak
Copy link
Collaborator Author

prusnak commented Apr 17, 2023

Converting to draft since it seems that gpt4all conversion is broken, see #984

Fix for this error is in #1031

GPT4All-J is not supported at the moment, which is reflected in the README

@prusnak
Copy link
Collaborator Author

prusnak commented Apr 18, 2023

Fixes #1036

@hughobrien
Copy link

suggest adding the SHASUMS so people can validate their conversions

@prusnak prusnak merged commit c6524f4 into master Apr 23, 2023
@prusnak prusnak deleted the readme-convert branch April 23, 2023 08:21
jeroen-mostert pushed a commit to jeroen-mostert/llama.cpp that referenced this pull request Aug 30, 2024
Co-authored-by: LostRuins Concedo <39025047+LostRuins@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants