-
Notifications
You must be signed in to change notification settings - Fork 248
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
Improve response formatting for SearchDocumentsTool #38
Comments
@philip-pentatonic Thanks for creating an issue! This happens when the LLM response is not correctly formatted, so we need to improve our output parsing on tools so they are correcting issues like this. PRs welcome from anyone who wants to take a look at this! 💙 |
My pleasure. I wish I could do more to help, but will raise issues as I find them. Have to say, this is the best repo I have seen so far on this. Really great work, thanks a lot for sharing it! |
I have my team chatting away (a mixture of amusement and wonderment!) and have a few errors in the terminal window. Not a good enough coder to fix them, but posting here if anyone else has them and/or wants to fix or advise:
Error: 1 validation error for SearchDocumentsToolInput
query
field required (type=value_error.missing)
The text was updated successfully, but these errors were encountered: