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

Fix: ollama long response timeout #3107

Merged

Conversation

TheTerrasque
Copy link

@TheTerrasque TheTerrasque commented Jun 12, 2024

Fix for #3067

Changes:

  • Added configurable timeout to aiohttp.ClientSession call to ollama, to work around default timeout of 5 minutes being too short in some situations
  • Set the new default timeout to 15 minutes (900 seconds)
  • Described new environment variable in TROUBLESHOOTING.md under title "Error on Slow Reponses for Ollama"

@TheTerrasque TheTerrasque changed the title Added timeout setting for ollama streaming response Fix: Added timeout setting for ollama streaming response Jun 12, 2024
@TheTerrasque TheTerrasque changed the title Fix: Added timeout setting for ollama streaming response Fix: ollama long response timeout Jun 12, 2024
@tjbck
Copy link
Contributor

tjbck commented Jun 13, 2024

This is great! Apologies for mistaking your bug report as one of the low quality ones 😅 I've just been getting thousands of emails a day and it's getting pretty hard to discern which ones are legitimate and which aren't (i'm working on this in my spare time, hope you understand 🙌: https://docs.openwebui.com/team). I'll rename the env var and merge the code. I appreciate the PR!

@tjbck
Copy link
Contributor

tjbck commented Jun 14, 2024

LGTM, Thanks!

@TheTerrasque
Copy link
Author

I understand, leading an open source project can be challenging. And thanks for considering the PR! Then I don't have to maintain a separate version 😄

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

2 participants