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

Update Dockerfile.buildkit #1542

Merged
merged 1 commit into from
Oct 21, 2023
Merged

Update Dockerfile.buildkit #1542

merged 1 commit into from
Oct 21, 2023

Conversation

tvhdev
Copy link
Contributor

@tvhdev tvhdev commented Oct 20, 2023

corrected model download directory

Describe your changes

the model download directory was wrong

Issue ticket number and link

Checklist before requesting a review

  • [X ] I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • I have added thorough documentation for my code.
  • I have tagged PR with relevant project labels. I acknowledge that a PR without labels may be dismissed.
  • If this PR addresses a bug, I have provided both a screenshot/video of the original bug and the working solution.

Demo

Steps to Reproduce

Notes

corrected model download directory

Signed-off-by: Thomas <tvhdev@vonhaugwitz-softwaresolutions.de>
@cebtenzzre
Copy link
Member

Why wouldn't we use GGUF models with the API? Everything in models2.json is in the GGUF directory.

@tvhdev
Copy link
Contributor Author

tvhdev commented Oct 20, 2023 via email

@cebtenzzre
Copy link
Member

Oh, I just noticed that this script is still set to use the old ggml-mpt-7b-chat.bin model. We need to update it to use a GGUF model instead.

@AndriyMulyar AndriyMulyar merged commit 34daf24 into nomic-ai:main Oct 21, 2023
@cebtenzzre
Copy link
Member

@AndriyMulyar Since you merged this, do you know how this works at all if it is using the old GGML format?

@AndriyMulyar
Copy link
Contributor

The documentation referencing ggML will work as long as the python version is pinned to a pre-2.x version.

@cebtenzzre
Copy link
Member

as long as the python version is pinned to a pre-2.x version

Where is it pinned? The gpt4all_api requirements.txt just lists gpt4all>=1.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants