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 local ollama doc #3966

Merged
merged 4 commits into from
Sep 20, 2024
Merged

Update local ollama doc #3966

merged 4 commits into from
Sep 20, 2024

Conversation

enyst
Copy link
Collaborator

@enyst enyst commented Sep 19, 2024

CHANGELOG

Update local Ollama docs. Since the app has changed, some settings have moved from files to the UI, and other tweaks.


Cc: @mamoodi

@mamoodi
Copy link
Collaborator

mamoodi commented Sep 20, 2024

Does this work if you follow the doc exactly as shown? Did you test it? Cause that would be great if we can refer to people to this doc just in case they want to test it.

@enyst
Copy link
Collaborator Author

enyst commented Sep 20, 2024

Does this work if you follow the doc exactly as shown? Did you test it? Cause that would be great if we can refer to people to this doc just in case they want to test it.

Of course. 🧸

Sorry, maybe it's not clear what's supposed to be happening here, let me put this differently:

  • as with other docs, we meet here the fact that we added API key and base URL to the UI. That implies that they're NOT needed anymore: 1) for running with docker run, with -e lines, 2) for running with make run, in config.toml. They won't hurt if they remain in these places too, but it will be confusing to people that they set them and they're now ignored. We can guide folks to set them in the UI settings instead.
  • specific to this doc, is that with docker run base url needs to be http://host.docker.internal:11434, while when running in dev mode with make run it needs to be `http://localhost:11434'.

@enyst enyst merged commit ebd9397 into main Sep 20, 2024
15 checks passed
@enyst enyst deleted the enyst/docs branch September 20, 2024 19:56
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