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

Fix some bugs in text2sql.(#4279)(#4281) #4280

Merged
merged 2 commits into from
Dec 30, 2024
Merged

Conversation

TeslaZY
Copy link
Contributor

@TeslaZY TeslaZY commented Dec 28, 2024

Fix some bugs in text2sql.(#4279)(#4281)

What problem does this PR solve?

  • The incorrect results in parsing CSV files of the QA knowledge base in the text2sql scenario. Process CSV files using the csv library. Decouple CSV parsing from TXT parsing
  • Most llm return results in markdown format sql query , Fix execution error caused by LLM output SQLmarkdown format.### Type of change
  • Bug Fix (non-breaking change which fixes an issue)

…base in the text2sql scenario.(#4279)

### What problem does this PR solve?
- The incorrect results in parsing CSV files of the QA knowledge base in the text2sql scenario.
- Process CSV files using the csv library.Decouple CSV parsing from TXT parsing

### Type of change
- [x] Bug Fix (non-breaking change which fixes an issue)
### What problem does this PR solve?
- most llm return results in markdown format ```sql query ```
- Dealing with the problem of sql execution error caused by LLM output SQLmarkdown format.

### Type of change
- [x] Bug Fix (non-breaking change which fixes an issue)
@TeslaZY TeslaZY changed the title Fixed the incorrect results in parsing CSV files of the QA knowledge for text2sql.(#4279) Fixed the some bugs in text2sql.(#4279) Dec 28, 2024
@TeslaZY TeslaZY changed the title Fixed the some bugs in text2sql.(#4279) Fixed the some bugs in text2sql.(#4279)(#4281) Dec 28, 2024
@TeslaZY TeslaZY changed the title Fixed the some bugs in text2sql.(#4279)(#4281) Fix some bugs in text2sql.(#4279)(#4281) Dec 28, 2024
@KevinHuSh KevinHuSh added the ci Continue Integration label Dec 30, 2024
@KevinHuSh KevinHuSh merged commit dd13a5d into infiniflow:main Dec 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continue Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants