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

Allow configureing the litellm Timeout #4995

Closed
IMbackK opened this issue Nov 14, 2024 · 5 comments
Closed

Allow configureing the litellm Timeout #4995

IMbackK opened this issue Nov 14, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@IMbackK
Copy link

IMbackK commented Nov 14, 2024

I seams (from the documentation) that there is no way to configure the litellm timeout, this is a problem as i am trying to use openahands with a local openai compatible server (tabbyai and aphrodite). Sometimes this can be very slow to return a compleation request, in this case openhands will timeout even though there is nothing wrong, the backend just needs more time.

@IMbackK IMbackK added the enhancement New feature or request label Nov 14, 2024
@mamoodi
Copy link
Collaborator

mamoodi commented Nov 14, 2024

If you are running the docker command, you can add:
-e SANDBOX_TIMEOUT=360

or whatever timeout you require to the command, and I believe it will work.

If you are running through the development workflow,
In the config.toml:
[sandbox]
timeout = 360

Let me know if this resolves your issue. I know for example if you wanted to install Java which takes a long time you have to modify this command.

If I misunderstood litellm timeout I apologize.

@neubig
Copy link
Contributor

neubig commented Nov 14, 2024

Hey @mamoodi , maybe it's actually LLM_TIMEOUT?

LLM_TIMEOUT = 'LLM_TIMEOUT'

@mamoodi
Copy link
Collaborator

mamoodi commented Nov 14, 2024

Ah thank you! Please try the above suggestion by passing in:
-e LLM_TIMEOUT=<your timeout>

to the docker command.

@IMbackK
Copy link
Author

IMbackK commented Nov 14, 2024

yes that seams to do the trick, i seams it might make sense to document the options provided by envvar via config.py

@IMbackK IMbackK closed this as completed Nov 14, 2024
@mamoodi
Copy link
Collaborator

mamoodi commented Nov 15, 2024

Yep we have an issue open for that! I believe someone is working on it. Thank you!

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

No branches or pull requests

3 participants