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

Add Workflow Executor Example #892

Merged
merged 12 commits into from
Nov 1, 2024

Conversation

JoshuaL3000
Copy link
Contributor

@JoshuaL3000 JoshuaL3000 commented Sep 30, 2024

Description

Add workflow executor example using existing react-langgraph strategy.

Issues

RFC: https://github.com/opea-project/docs/pull/77/files

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)

@xuechendi xuechendi marked this pull request as draft September 30, 2024 14:34
JoshuaL3000 and others added 3 commits October 8, 2024 09:53
* Use single tool implementation
* Add CICD validation script
* Update example readme

Signed-off-by: JoshuaL3000 <[email protected]>
@JoshuaL3000
Copy link
Contributor Author

Hi @xuechendi, the CICD validation test can only pass after the addition of custom system prompts and the merge for opea-project/GenAIComps#782

Copy link
Collaborator

@xuechendi xuechendi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xuechendi
Copy link
Collaborator

@JoshuaL3000 , codes looks good to me, I was discussing with Minmin on your previous WorkExecutor strategy. Once this example is merged, we can't continue on that strategy

@chensuyue chensuyue mentioned this pull request Oct 24, 2024
4 tasks
@JoshuaL3000 JoshuaL3000 marked this pull request as ready for review October 25, 2024 07:32
@JoshuaL3000
Copy link
Contributor Author

Hi @xuechendi @minmin-intel, the CICD results are passing. Could you help to review those and see if this is ready to be merged? thanks

@xuechendi
Copy link
Collaborator

Result is looking good
image

@xuechendi xuechendi merged commit bf5c391 into opea-project:main Nov 1, 2024
14 checks passed
arcyleung pushed a commit to arcyleung/GenAIExamples that referenced this pull request Nov 4, 2024
Signed-off-by: JoshuaL3000 <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: Arthur Leung <[email protected]>
astafevav pushed a commit to astafevav/GenAIExamples that referenced this pull request Nov 6, 2024
Signed-off-by: JoshuaL3000 <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: astafevav <[email protected]>
wangkl2 pushed a commit to wangkl2/GenAIExamples that referenced this pull request Dec 11, 2024
* Embedding TEI Langchain compatible with OpenAI API

Signed-off-by: Xinyao Wang <[email protected]>

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

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

* TextDoc support list

Signed-off-by: Xinyao Wang <[email protected]>

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

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

* support tei llama index openai compatible API

Signed-off-by: Xinyao Wang <[email protected]>

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

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

* support mosec langchain openai compatible API

Signed-off-by: Xinyao Wang <[email protected]>

* update UT for embedding tests

Signed-off-by: Xinyao Wang <[email protected]>

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

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

* fix ut bug

Signed-off-by: Xinyao Wang <[email protected]>

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

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

* support embedding predictionguard  openai compatible API

Signed-off-by: Xinyao Wang <[email protected]>

* support embedding multimodal clip OpenAI compatible API

Signed-off-by: Xinyao Wang <[email protected]>

* fix bug

Signed-off-by: Xinyao Wang <[email protected]>

* enable debug mode for embedding UT

Signed-off-by: Xinyao Wang <[email protected]>

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

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

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

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

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

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

---------

Signed-off-by: Xinyao Wang <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: chen, suyue <[email protected]>
Co-authored-by: ZePan110 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants