-
Notifications
You must be signed in to change notification settings - Fork 47
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
SQL error: execute() first #7
Comments
It means your database is not connected
…On Mon, 17 Jul 2023 at 6:14 PM, TianMin ***@***.***> wrote:
[image: image]
<https://user-images.githubusercontent.com/10555409/253911188-a4906d0c-5071-45ee-869a-07c46d3c2f86.png>
—
Reply to this email directly, view it on GitHub
<#7>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADDKTMVPCQ47PC7XG3GYHQTXQUF6VANCNFSM6AAAAAA2MW4OJY>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Change the pattern to: |
Hello, author. We followed the advice and modified the pattern of the regular expression. However, this caused all the answers to the questions to include the phrase "not need memory". We suspect that there might still be an issue with the pattern here. @potong |
I try to ask the question "What was the total revenue for January 2023?", The LLM generates the following middle-stage answer:
Notice that, the SQL is not surrounded by backtick
|
The text was updated successfully, but these errors were encountered: