-
Notifications
You must be signed in to change notification settings - Fork 720
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
Timeout during /improve
command
#178
Comments
Hey @okotek were y'all able to get this - it's the Happy to make a PR if required. |
Was addressed in #181 as far as I can tell, although even with 180 seconds I'm still seeings timeouts with large PRs. I'll try creating a reproducible example. |
Hmm.. maybe we can remove timeouts from lite / make that an optional parameter, this should prevent these scenarios from occurring. This was primarily added to deal with Azure / Replicate related issues where the model calls would hang for an unusual amount of time. |
Ok, tracking this on our end - BerriAI/litellm#73 Will aim to have an updated version tested + pushed out by EOD. I'll make a PR for the change, which can then run through the normal testing y'all have for PR-Agent as well. |
It's possible that this is the issue I'm facing since I'm using Azure. I've only ever noticed it happen with |
I'm guessing pre-lite it didn't timeout, so did it hang or always complete successfully? |
From what I can tell it always completed successfully, but then again this is all pretty new stuff so it's possible I just didn't run into a large enough PR when using When I get the timeout again I'll try to run the pre-lite agent on the same PR and see if it works. |
sounds great! i'll put set_timeouts as an optional param, which should remove any lite-related blockers between you-your azure calls. |
update: default timeouts are now set to 600 seconds (~10 minutes) |
update: tested |
/similar_issue |
I'm getting a timeout error when calling
/improve
. This seems to happen when the PR diff is fairly large. For example: 11 files changed, ~1k additions, ~300 deletions. In myconfiguration.toml
I setpr_code_suggestions.num_code_suggestions=5
The text was updated successfully, but these errors were encountered: