-
Notifications
You must be signed in to change notification settings - Fork 301
[Pending] Merge Translation and CodeTrans examples #1670
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
Conversation
…compose of rocm and xeon part Signed-off-by: letonghan <[email protected]>
Dependency Review✅ No vulnerabilities or license issues found.Scanned Files |
for more information, see https://pre-commit.ci
…into merge_translation
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
Signed-off-by: letonghan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still have codetrans e2e example after this merge? Or the codetrans e2e will be gone?
|
||
``` | ||
export HFTOKEN="insert-your-huggingface-token-here" | ||
helm install codetrans oci://ghcr.io/opea-project/charts/codetrans --set global.HUGGINGFACEHUB_API_TOKEN=${HFTOKEN} -f gaudi-values.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only works with codetrans, won't work with translation. We need to create a new helm chart for translation after this PR is merged. I would suggest keep the helm chart part empty here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like we should leave this part empty, and the helm chart of these two examples should be merged into one
The CodeTrans example will be removed, and only the Translation example exists in GenAIExample. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's hold off on merging this PR to v1.3 for now.
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
This PR was closed because it has been stalled for 7 days with no activity. |
Description
Refactor Translation example and merge with CodeTrans.
Update python files, READMEs, docker compose files and k8s files.
The endpoint of megaservice is unified to
/v1/translation
, and add an additional parametertranslation_type
for different llm prompt.Issues
#1240
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
None
Tests
Describe the tests that you ran to verify your changes.