-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: more graceful handling of API call didn't return a message
message.
#1796
Comments
@sarahwooders Thank you for capturing this. I'm happy to provide context, and have some suggestions:
Suggestion: I wish it was more clear in the UX that Agents override the model from the config. In fact, nowhere on the Agents dashboard or Agent Chat UIs does it mention what model is being used. Expanding an Agent in the dashboard only shows |
regarding this message sometimes I got the impression that this error / warning came even though a return message was actually returned (because I see the returned message in the stack trace) |
When the model fails to generate a message for
send_message
, we should re-try or give a better errror:The text was updated successfully, but these errors were encountered: