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

S14 bug missing tool message (merge messages) #211

Merged
merged 2 commits into from
Jun 20, 2024

Conversation

anthony2261
Copy link
Collaborator

Merged AIMessage into ToolMessage. Reworded message content to sound more like a tool message rather than an AI message. Briefly tested change with messages asking for multiple plots, multiple sample rows from different tables, etc.

@anthony2261 anthony2261 linked an issue Jun 19, 2024 that may be closed by this pull request
@RamiAwar RamiAwar merged commit c953ffb into main Jun 20, 2024
4 checks passed
@RamiAwar RamiAwar deleted the S14-bug-missing-tool-message-anthony branch June 20, 2024 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with Parallel Tool calling the QuerySQLDataBaseTool
2 participants