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

Charts getting linked to wrong query #215

Open
RamiAwar opened this issue Jun 20, 2024 · 0 comments
Open

Charts getting linked to wrong query #215

RamiAwar opened this issue Jun 20, 2024 · 0 comments
Assignees
Labels
bug Something isn't working multi-question Linked to multi-question results (ex. 2 charts)

Comments

@RamiAwar
Copy link
Owner

Current logic of result linking links to the latest generated query. This apparently is not enough, parallel calls do happen sometimes, and that leads to bad linking which is very hard to correct.

Ideas:

  • create an ID on each sql query, and ask the LLM to pass in this ID to the chart generation tool as an input. This ID will be used to link the two (this way we know which chart corresponds to which query)

  • create a subgraph for chart generation that comes after an SQL query gen tool call

  • link bas on time (1st sql with for_chart corresponds to 1st chart, 2nd sql with for_chart to 2nd chart, etc.)

@RamiAwar RamiAwar added the bug Something isn't working label Jun 20, 2024
@RamiAwar RamiAwar self-assigned this Jun 20, 2024
@RamiAwar RamiAwar added the multi-question Linked to multi-question results (ex. 2 charts) label Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working multi-question Linked to multi-question results (ex. 2 charts)
Projects
None yet
Development

No branches or pull requests

1 participant