Add SQL agent strategy by minmin-intel · Pull Request #975 · opea-project/GenAIComps

minmin-intel · 2024-12-06T00:43:37Z

Description

Add sql_agent_llama strategy

Issues

#976

Type of change

List the type of change like below. Please delete options that are not relevant.

New feature (non-breaking change which adds new functionality)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

Signed-off-by: minmin-intel <minmin.hou@intel.com>

for more information, see https://pre-commit.ci

Signed-off-by: minmin-intel <minmin.hou@intel.com>

…IComps into sql-agent-v1.2

for more information, see https://pre-commit.ci

Signed-off-by: minmin-intel <minmin.hou@intel.com>

…IComps into sql-agent-v1.2

for more information, see https://pre-commit.ci

minmin-intel · 2024-12-06T22:27:39Z

UT tests sql_agent_llama on Gaudi.
I tested in my local system using openai model for sql_agent and passed. See screenshot below.

Signed-off-by: minmin-intel <minmin.hou@intel.com>

…IComps into sql-agent-v1.2

for more information, see https://pre-commit.ci

xuechendi

LGTM

lkk12014402

LGTM

Signed-off-by: minmin-intel <minmin.hou@intel.com>

for more information, see https://pre-commit.ci

Signed-off-by: minmin-intel <minmin.hou@intel.com>

for more information, see https://pre-commit.ci

Signed-off-by: minmin-intel <minmin.hou@intel.com>

…IComps into sql-agent-v1.2

for more information, see https://pre-commit.ci

Signed-off-by: minmin-intel <minmin.hou@intel.com>

…IComps into sql-agent-v1.2

Signed-off-by: minmin-intel <minmin.hou@intel.com>

…IComps into sql-agent-v1.2

Signed-off-by: minmin-intel <minmin.hou@intel.com>

for more information, see https://pre-commit.ci

Signed-off-by: minmin-intel <minmin.hou@intel.com>

ftian1 · 2024-12-12T01:54:13Z

this PR is not correct. it introduces a symblic link in root dir and add a data_hub. it's not allowed. pls revert it @minmin-intel

This reverts commit c36c503.

This reverts commit c36c503. Co-authored-by: lkk <33276950+lkk12014402@users.noreply.github.com>

* initial code for sql agent llama Signed-off-by: minmin-intel <minmin.hou@intel.com> * add test for sql agent Signed-off-by: minmin-intel <minmin.hou@intel.com> * update sql agent test Signed-off-by: minmin-intel <minmin.hou@intel.com> * fix bugs and use vllm to test sql agent Signed-off-by: minmin-intel <minmin.hou@intel.com> * add tag-bench test and google search tool Signed-off-by: minmin-intel <minmin.hou@intel.com> * test sql agent with hints Signed-off-by: minmin-intel <minmin.hou@intel.com> * fix bugs for sql agent with hints and update test Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * add readme for sql agent and fix ci bugs Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * add sql agent using openai models Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fix bugs in sql agent openai Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * make wait time longer for sql agent microservice to be ready Signed-off-by: minmin-intel <minmin.hou@intel.com> * update readme Signed-off-by: minmin-intel <minmin.hou@intel.com> * fix test bug Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * skip planexec with vllm due to vllm-gaudi bug Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * debug ut issue Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * use vllm for all uts Signed-off-by: minmin-intel <minmin.hou@intel.com> * debug ci issue Signed-off-by: minmin-intel <minmin.hou@intel.com> * change vllm port Signed-off-by: minmin-intel <minmin.hou@intel.com> * update ut Signed-off-by: minmin-intel <minmin.hou@intel.com> * remove tgi server Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * align vllm port Signed-off-by: minmin-intel <minmin.hou@intel.com> --------- Signed-off-by: minmin-intel <minmin.hou@intel.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> opea bedrock integration Signed-off-by: vihanth sura <vihanth@amazon.com>

This reverts commit c36c503. Co-authored-by: lkk <33276950+lkk12014402@users.noreply.github.com>

* initial code for sql agent llama Signed-off-by: minmin-intel <minmin.hou@intel.com> * add test for sql agent Signed-off-by: minmin-intel <minmin.hou@intel.com> * update sql agent test Signed-off-by: minmin-intel <minmin.hou@intel.com> * fix bugs and use vllm to test sql agent Signed-off-by: minmin-intel <minmin.hou@intel.com> * add tag-bench test and google search tool Signed-off-by: minmin-intel <minmin.hou@intel.com> * test sql agent with hints Signed-off-by: minmin-intel <minmin.hou@intel.com> * fix bugs for sql agent with hints and update test Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * add readme for sql agent and fix ci bugs Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * add sql agent using openai models Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fix bugs in sql agent openai Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * make wait time longer for sql agent microservice to be ready Signed-off-by: minmin-intel <minmin.hou@intel.com> * update readme Signed-off-by: minmin-intel <minmin.hou@intel.com> * fix test bug Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * skip planexec with vllm due to vllm-gaudi bug Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * debug ut issue Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * use vllm for all uts Signed-off-by: minmin-intel <minmin.hou@intel.com> * debug ci issue Signed-off-by: minmin-intel <minmin.hou@intel.com> * change vllm port Signed-off-by: minmin-intel <minmin.hou@intel.com> * update ut Signed-off-by: minmin-intel <minmin.hou@intel.com> * remove tgi server Signed-off-by: minmin-intel <minmin.hou@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * align vllm port Signed-off-by: minmin-intel <minmin.hou@intel.com> --------- Signed-off-by: minmin-intel <minmin.hou@intel.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

This reverts commit e9f95ab. Co-authored-by: lkk <33276950+lkk12014402@users.noreply.github.com>

minmin-intel added 7 commits December 3, 2024 15:17

initial code for sql agent llama

bd44bb6

Signed-off-by: minmin-intel <minmin.hou@intel.com>

add test for sql agent

26d7964

Signed-off-by: minmin-intel <minmin.hou@intel.com>

update sql agent test

ad4969a

Signed-off-by: minmin-intel <minmin.hou@intel.com>

fix bugs and use vllm to test sql agent

a6ef75d

Signed-off-by: minmin-intel <minmin.hou@intel.com>

add tag-bench test and google search tool

a63d3bc

Signed-off-by: minmin-intel <minmin.hou@intel.com>

test sql agent with hints

c1ca4b4

Signed-off-by: minmin-intel <minmin.hou@intel.com>

fix bugs for sql agent with hints and update test

cc56a6c

Signed-off-by: minmin-intel <minmin.hou@intel.com>

minmin-intel requested a review from lvliang-intel as a code owner December 6, 2024 00:43

minmin-intel requested review from lkk12014402 and xuechendi and removed request for lvliang-intel December 6, 2024 00:43

pre-commit-ci bot and others added 8 commits December 6, 2024 00:44

[pre-commit.ci] auto fixes from pre-commit.com hooks

d063bf9

for more information, see https://pre-commit.ci

Merge branch 'main' into sql-agent-v1.2

6191bd9

add readme for sql agent and fix ci bugs

1f207b1

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'sql-agent-v1.2' of https://github.com/minmin-intel/GenA…

eb692b1

…IComps into sql-agent-v1.2

[pre-commit.ci] auto fixes from pre-commit.com hooks

48ca517

for more information, see https://pre-commit.ci

add sql agent using openai models

66c27e2

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'sql-agent-v1.2' of https://github.com/minmin-intel/GenA…

fde2d72

…IComps into sql-agent-v1.2

[pre-commit.ci] auto fixes from pre-commit.com hooks

a8897b5

for more information, see https://pre-commit.ci

minmin-intel and others added 3 commits December 6, 2024 15:01

fix bugs in sql agent openai

a93f0d4

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'sql-agent-v1.2' of https://github.com/minmin-intel/GenA…

c154f84

…IComps into sql-agent-v1.2

[pre-commit.ci] auto fixes from pre-commit.com hooks

b789495

for more information, see https://pre-commit.ci

xuechendi approved these changes Dec 7, 2024

View reviewed changes

lkk12014402 approved these changes Dec 7, 2024

View reviewed changes

minmin-intel and others added 5 commits December 9, 2024 18:46

make wait time longer for sql agent microservice to be ready

0644105

Signed-off-by: minmin-intel <minmin.hou@intel.com>

update readme

48935b0

Signed-off-by: minmin-intel <minmin.hou@intel.com>

fix test bug

b86bbea

Signed-off-by: minmin-intel <minmin.hou@intel.com>

[pre-commit.ci] auto fixes from pre-commit.com hooks

88c796d

for more information, see https://pre-commit.ci

skip planexec with vllm due to vllm-gaudi bug

7622cfd

Signed-off-by: minmin-intel <minmin.hou@intel.com>

pre-commit-ci bot and others added 15 commits December 9, 2024 22:33

[pre-commit.ci] auto fixes from pre-commit.com hooks

5f76d82

for more information, see https://pre-commit.ci

debug ut issue

b80ee58

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'sql-agent-v1.2' of https://github.com/minmin-intel/GenA…

ac0fc86

…IComps into sql-agent-v1.2

[pre-commit.ci] auto fixes from pre-commit.com hooks

9d85cb3

for more information, see https://pre-commit.ci

use vllm for all uts

b5c0e2a

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'sql-agent-v1.2' of https://github.com/minmin-intel/GenA…

e02b4cd

…IComps into sql-agent-v1.2

debug ci issue

4b239ec

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'sql-agent-v1.2' of https://github.com/minmin-intel/GenA…

3c1a220

…IComps into sql-agent-v1.2

change vllm port

bb0d24d

Signed-off-by: minmin-intel <minmin.hou@intel.com>

update ut

f5e29d9

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'main' into sql-agent-v1.2

faeeeb6

remove tgi server

8dfccaa

Signed-off-by: minmin-intel <minmin.hou@intel.com>

[pre-commit.ci] auto fixes from pre-commit.com hooks

1feb779

for more information, see https://pre-commit.ci

align vllm port

14c61f9

Signed-off-by: minmin-intel <minmin.hou@intel.com>

Merge branch 'main' into sql-agent-v1.2

880fe43

minmin-intel merged commit c36c503 into opea-project:main Dec 11, 2024

joshuayao linked an issue Dec 12, 2024 that may be closed by this pull request

Add sql agent strategy #976

Closed

minmin-intel added a commit that referenced this pull request Dec 12, 2024

Revert "Add SQL agent strategy (#975)"

906bfc8

This reverts commit c36c503.

minmin-intel mentioned this pull request Dec 12, 2024

Revert "Add SQL agent strategy" #1030

Merged

lvliang-intel pushed a commit that referenced this pull request Dec 14, 2024

Revert "Add SQL agent strategy (#975)" (#1030)

46835f9

This reverts commit c36c503. Co-authored-by: lkk <33276950+lkk12014402@users.noreply.github.com>

jjmaturino pushed a commit to jjmaturino/GenAIComps that referenced this pull request Dec 16, 2024

Revert "Add SQL agent strategy (opea-project#975)" (opea-project#1030)

2f43629

This reverts commit c36c503. Co-authored-by: lkk <33276950+lkk12014402@users.noreply.github.com>

smguggen pushed a commit to opea-aws-proserve/GenAIComps that referenced this pull request Jan 23, 2025

Revert "Add SQL agent strategy (opea-project#975)" (opea-project#1030)

a04b2a2

This reverts commit c36c503. Co-authored-by: lkk <33276950+lkk12014402@users.noreply.github.com>

madison-evans pushed a commit to SAPD-Intel/GenAIComps that referenced this pull request May 12, 2025

Revert "Add SQL agent strategy (opea-project#975)" (opea-project#1030)

7824358

This reverts commit e9f95ab. Co-authored-by: lkk <33276950+lkk12014402@users.noreply.github.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add SQL agent strategy#975

Add SQL agent strategy#975
minmin-intel merged 38 commits intoopea-project:mainfrom
minmin-intel:sql-agent-v1.2

minmin-intel commented Dec 6, 2024 •

edited

Loading

Uh oh!

minmin-intel commented Dec 6, 2024

Uh oh!

xuechendi left a comment

Uh oh!

lkk12014402 left a comment

Uh oh!

ftian1 commented Dec 12, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

minmin-intel commented Dec 6, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Issues

Type of change

Dependencies

Tests

Uh oh!

minmin-intel commented Dec 6, 2024

Uh oh!

xuechendi left a comment

Choose a reason for hiding this comment

Uh oh!

lkk12014402 left a comment

Choose a reason for hiding this comment

Uh oh!

ftian1 commented Dec 12, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

minmin-intel commented Dec 6, 2024 •

edited

Loading