Skip to content

Add custom client earlier in README #486

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

byarbrough
Copy link

@byarbrough byarbrough commented Mar 28, 2025

Without this it's not immediately apparent that the library is making HTTP requests, or to where it might be making them.

Without this it's opaque that the library is making HTTP requests,
or to where it might be making them.

The only place I could find how to change the localhost:11434 behavior
was in the Ollama FAQs. If there is a way to set it besides OLLAMA_HOST I would love to know!
@byarbrough
Copy link
Author

byarbrough commented Mar 28, 2025

Yikes. I just saw the custom client section below. Going to update this.
Ok, ready

@byarbrough byarbrough changed the title Add OLLAMA_HOST blurb to README Add custom client earlier in README Mar 28, 2025
@ParthSareen
Copy link
Member

I'm okay with the callout to The library connects to Ollama running on http://localhost:11434` by default.` but it should be moved up to right under the description.

I'd rather not add the host callouts in here as they are more Ollama related and not directly the library. I think it would be better to add a link to https://github.com./ollama/ollama/blob/main/docs/faq.md instead.

Comment on lines +12 to +13
You can change this address with the [custom client](https://github.com./ollama/ollama-python?tab=readme-ov-file#custom-client)
or by setting the [`OLLAMA_HOST`](https://github.com./ollama/ollama/blob/main/docs/faq.md#how-can-i-expose-ollama-on-my-network) environment variable.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
You can change this address with the [custom client](https://github.com./ollama/ollama-python?tab=readme-ov-file#custom-client)
or by setting the [`OLLAMA_HOST`](https://github.com./ollama/ollama/blob/main/docs/faq.md#how-can-i-expose-ollama-on-my-network) environment variable.

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.

2 participants