fix: multi-round conversion with Bedrock Claude

Sinaptik-AI / pandas-ai

Chat with your database (SQL, CSV, pandas, polars, mongodb, noSQL, etc). PandasAI makes data analysis conversational using LLMs (GPT 3.5 / 4, Anthropic, VertexAI) and RAG.

https://pandas-ai.com

Other

12.73k stars 1.23k forks source link

fix: multi-round conversion with Bedrock Claude #1111

Closed goriri closed 5 months ago

goriri commented 5 months ago

[x] Tests added and passed if fixing a bug or adding a new feature.
[x] All code checks passed.

gventuri commented 5 months ago

@goriri can you provide slightly more context about this fix? I don't understand exactly why we are overriding the previous message in the conversation. Does bedrock raise an error if there are 2 user messages in a row?

gventuri commented 5 months ago

@goriri let me know, so we can merge :)

goriri commented 5 months ago

@gventuri yes. it will complain when two user messages in a row (requiring alternating user/assistant messages). in a single message, the content can contain multiple text sections.

gventuri commented 5 months ago

Thanks a lot for letting me know, merging! @goriri