Skip to content

Add Post-SQL routing step #16

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.

Sign up for GitHub

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

Jump to bottom
Merged
vrtornisiello merged 16 commits into from
May 29, 2025
Merged

Add Post-SQL routing step #16

vrtornisiello merged 16 commits into from
May 29, 2025

Conversation

@vrtornisiello
Copy link
Collaborator

@vrtornisiello vrtornisiello commented May 28, 2025

  • Added a node for checking if the SQL Agent's answer would benefit of a graphical visualization before calling the Viz Agent, for reducing costs and latency.
  • Updated pyproject.toml and poetry.lock files.
  • Updated CI workflow.

@vrtornisiello
update dependencies files
d534329
@vrtornisiello
created a node to decide if the viz agent should be called
be155f9
@vrtornisiello
tweaked routing system prompts and structured outputs
e767a99
@vrtornisiello
refined router agent and fixed final answer processing
bd7c4fd
@vrtornisiello
using .get to avoid KeyError
853db89
@vrtornisiello
add grpcio to the dependencies
b967113
@vrtornisiello
check when there is only one query result
f5ba910
@vrtornisiello
bump version to 0.4.3
b3f0cda
@vrtornisiello
tweak post sql router system prompt
e60bf7e
@vrtornisiello
added the workflow_dispatch option for manually triggering actions …
4001303 
…and unified and simplified the installation and test steps
@vrtornisiello
update poetry lock file
bf65977
@vrtornisiello
fix RouterAgent and update test cases
441fb77
@vrtornisiello
dependency installation and tests execution separated into two steps
7b768bf
@vrtornisiello
return an empty Chart object when the VizAgent is not called, oth…
bb18b12 
…erwise the chart from the last run will be returned
@vrtornisiello
update docstring
0a9ba35
@vrtornisiello
update test case
f58ac31
@vrtornisiello vrtornisiello merged commit 20c0432 into main May 29, 2025
1 check passed
@vrtornisiello vrtornisiello deleted the check_before_viz branch May 29, 2025 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vrtornisiello