-
Notifications
You must be signed in to change notification settings - Fork 73
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
Handle Anthropic overloaded errors during streaming #193
Comments
Just saw this myself this morning! Thanks for reporting it. |
There's more to handling this than I've done so far. It helps that they keep making it reproducible. 🤦♂️ |
I haven't had a chance to pull your changes and do my own testing but I will here shortly. It's unfortunate how frequently their API is overloaded recently. |
Yeah. I’m going to take a different approach. I’ll try raising an exception
and catching it in the LLMChain.
So no, it’s not working great yet.
…On Fri, Nov 22, 2024 at 7:37 AM Sean Callan ***@***.***> wrote:
I haven't had a chance to pull your changes and do my own testing but I
will here shortly.
It's unfortunate how frequently their API is overloaded recently.
—
Reply to this email directly, view it on GitHub
<#193 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGFQGGHCD4C2EKJRPPBGJT2B46TPAVCNFSM6AAAAABSHHUKYSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJTHEYDSMBWHA>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
@doomspork I pushed up a hopefully more complete fix. When it happens again, the Where the error has a If the pattern holds, it will be down again tomorrow morning (but it's the weekend, so perhaps not). I'll be watching for it too though. |
Sounds good @brainlid! Thanks for tackling this one 🙏 I'll let you know what I see 🫡 |
As we're making more use of streaming we're seeing more of these errors recently:
The result is this error logged but the code seems to proceed like everything is okay. I would expect/prefer an error were raised so we could retry the conversation versus leaving it in a partial and incomplete state.
I started spelunking today but it looks like handling this could be a decent sized changes so I thought it was best to bring it here for discussion.
Anthropic documentation around streaming errors for reference: https://docs.anthropic.com/en/api/messages-streaming#error-events
The text was updated successfully, but these errors were encountered: