langchain icon indicating copy to clipboard operation
langchain copied to clipboard

fix: InfoSQLDatabaseTool on openai functions mode sometimes calling tables instead of table_names as function arg

Open puigde opened this issue 2 years ago • 5 comments

Description: Improve the description for the InfoSQLDatabaseTool in order to ensure that the llm guesses the input properly on openai functions mode.

puigde avatar Nov 28 '23 20:11 puigde

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Jan 25, 2024 3:57pm

vercel[bot] avatar Nov 28 '23 20:11 vercel[bot]

thanks for pr @puigde! for context, what's openai function output you're seeing?

baskaryan avatar Nov 29 '23 00:11 baskaryan

thanks for pr @puigde! for context, what's openai function output you're seeing?

Hi @baskaryan it depended but most times the agent tried to pass the argument as 'tables' instead of 'table_names' raising a invalid args for the function call.

puigde avatar Dec 09 '23 21:12 puigde

Any updates on this?

AlexCapchase avatar Jan 16 '24 15:01 AlexCapchase

Any updates on this? Looks like sql tools stuff had a path change bringing merge conflicts. Now should be ok for merge afaik. cc. @baskaryan, @hwchase17 ??

puigde avatar Jan 17 '24 12:01 puigde

resolved in #16756

baskaryan avatar Mar 30 '24 01:03 baskaryan