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

code-gen-react-ui #417

Merged
merged 29 commits into from
Jul 24, 2024
Merged

Conversation

jaswanth8888
Copy link
Collaborator

Description

Added Code Gen React UI

Issues

n/a.

Type of change

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)

Dependencies

  • mantine/core
  • mantine/hooks
  • microsoft/fetch-event-source
  • reduxjs/toolkit
  • tabler/icons-react
  • axios
  • react
  • react-dom
  • react-markdown
  • react-syntax-highlighter
  • remark-frontmatter
  • remark-gfm

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

hteeyeoh and others added 2 commits July 17, 2024 15:40
Include manifest to deploy CodeGen pipeline with ReactUI into
Kubernetes. Include as well README file for the steps.

Signed-off-by: Yeoh, Hoong Tee <[email protected]>
Signed-off-by: jaswanth8888 <[email protected]>
@chensuyue
Copy link
Collaborator

Add one line after this one to replace the registry,

sed -i "s#image: opea/codegen-ui:latest#image: opea/codegen-ui:${IMAGE_TAG}#g" docker_compose.yaml

@jaswanth8888
Copy link
Collaborator Author

sed -i "s#image: opea/codegen-ui:latest#image: opea/codegen-ui:${IMAGE_TAG}#g" docker_compose.yaml

added

@chensuyue chensuyue added this to the v0.8 milestone Jul 19, 2024
@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

If this works in your local test, I will do more test to identify the issue.

@yogeshmpandey
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

If this works in your local test, I will do more test to identify the issue.

@chensuyue

Tests works locally on Xeon.

image

CodeGen/docker/ui/react/README.md Outdated Show resolved Hide resolved
CodeGen/docker/ui/react/public/vite.svg Outdated Show resolved Hide resolved
CodeGen/docker/ui/react/src/vite-env.d.ts Outdated Show resolved Hide resolved
CodeGen/docker/xeon/docker_compose.yaml Show resolved Hide resolved
CodeGen/docker/gaudi/docker_compose.yaml Show resolved Hide resolved
@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

If this works in your local test, I will do more test to identify the issue.

@chensuyue

Tests works locally on Xeon.

image

Ok, then we will do some local test on CI machine.

Signed-off-by: jaswanth8888 <[email protected]>
@jaswanth8888 jaswanth8888 requested a review from WenjiaoYue July 23, 2024 09:48
@jaswanth8888 jaswanth8888 requested a review from chensuyue July 24, 2024 07:35
Signed-off-by: jaswanth8888 <[email protected]>
@jaswanth8888 jaswanth8888 requested a review from chensuyue July 24, 2024 08:07
@jaswanth8888 jaswanth8888 merged commit 1b48e54 into opea-project:main Jul 24, 2024
19 of 20 checks passed
yogeshmpandey pushed a commit to hteeyeoh/GenAIExamples that referenced this pull request Aug 12, 2024
* CodeGen: Manifest to deploy CodeGen with ReactUI into Kubernetes

Include manifest to deploy CodeGen pipeline with ReactUI into
Kubernetes. Include as well README file for the steps.

Signed-off-by: Yeoh, Hoong Tee <[email protected]>

* code-gen-react-ui

Signed-off-by: jaswanth8888 <[email protected]>

* made changes as per PR suggestions

Signed-off-by: jaswanth8888 <[email protected]>

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

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

* updated readme

Signed-off-by: jaswanth8888 <[email protected]>

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

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

* update base image

Signed-off-by: jaswanth8888 <[email protected]>

* updated codegn dockerfile react

Signed-off-by: jaswanth8888 <[email protected]>

* updated wait time

Signed-off-by: jaswanth8888 <[email protected]>

* updated as per PR comments

Signed-off-by: jaswanth8888 <[email protected]>

* aded react ui in gaudi test file

Signed-off-by: jaswanth8888 <[email protected]>

* fixed PR comments

Signed-off-by: jaswanth8888 <[email protected]>

* added docker pull for text-generation-model

Signed-off-by: jaswanth8888 <[email protected]>

* removed unused files

Signed-off-by: jaswanth8888 <[email protected]>

---------

Signed-off-by: Yeoh, Hoong Tee <[email protected]>
Signed-off-by: jaswanth8888 <[email protected]>
Co-authored-by: Yeoh, Hoong Tee <[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]>
wangkl2 pushed a commit to wangkl2/GenAIExamples that referenced this pull request Dec 11, 2024
…ce (opea-project#417)

* remrove deprecated imports and switch to langchain-huggingface

Signed-off-by: rbrugaro <[email protected]>

* fix typo

Signed-off-by: rbrugaro <[email protected]>

* set requirements to langchain latest release

Signed-off-by: rbrugaro <[email protected]>

* remove version for langchain in requirements

Signed-off-by: rbrugaro <[email protected]>

---------

Signed-off-by: rbrugaro <[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.

6 participants