search

Canner / WrenAI

🚀 Open-source SQL AI Agent for Text-to-SQL. Make Text2SQL Easy! 🙌
https://getwren.ai/oss
GNU Affero General Public License v3.0
1.73k stars 155 forks source link

remove query understanding #463

Closed cyyeh closed 2 months ago

cyyeh commented 3 months ago

since the functionality of the "query understanding" pipeline is unneccesary(it's only aimed to refuse user's questions using random words) at the moment, and I found some models may generate incorrect responses at this pipeline. In the future, we may add it back with much better design for user's query clarification.

cyyeh commented 2 months ago

overall LGTM. only have a question about should we also remove the error code for the misleading query?

https://github.com/Canner/WrenAI/blob/e97a9de73a87881b500b4a0269eef11ade5bc04a/wren-ai-service/src/web/v1/services/ask.py#L94-L98

I added the comment instead: MISLEADING_QUERY is not in use now, we may add it back in the future when we implement the clarification pipeline