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

feature: add llama api using local models #108

Closed
wants to merge 8 commits into from

Conversation

mkellerman
Copy link

This allows the user to un-comment the section in the docker-compose.yml file and .env to either use the mocked-api or the llama-api.

The user can then load local models, and use chatgpt-web against it.

@Niek
Copy link
Owner

Niek commented Apr 11, 2023

Thanks for your contribution!

I'm a bit in doubt which approach to take: your PR or what @mudler proposed in #105 - I'm leaning toward his proposal because it's compatible with more models, and since the API is hosted in a separate repo/docker image it should be better updated.

Any thoughts?

@Niek
Copy link
Owner

Niek commented Apr 11, 2023

I see that you actually worked in this in https://github.com/mkellerman/chatgpt-web/tree/feature/add-llama-cli, great :)

@mkellerman
Copy link
Author

Yes, weither approaches would work. I dont see a negative impact to have both repo document/showcase one another. And it allows both projects to grow and thrive.

@mkellerman
Copy link
Author

Having it hear helps with the development work of the UI. And having it overthere helps people working on their MLs.

@Niek
Copy link
Owner

Niek commented Apr 12, 2023

OK, I will support both - but first I will contribute a Dockerfile to the llama-cpp-python repo, so we can just pull the latest version and not maintain that in this repo.

Edit: done, see abetlen/llama-cpp-python#73

@countzero
Copy link

@mkellerman & @Niek I am not sure, if I am missing anything, but isn't this capability already implemented by #494?

You can simply start a local OpenAI API compatible llama-server hosting a local model and point chatgpt-web against it:

image
image
image

@Niek
Copy link
Owner

Niek commented Oct 10, 2024

Yes, this is an old issue that can be closed now. PRs are welcome with some docs around this though :)

@Niek Niek closed this Oct 10, 2024
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.

3 participants