Skip to content

[ Translation ] Refine documents#1795

Merged
lvliang-intel merged 17 commits into
mainfrom
doc-translation
Apr 20, 2025
Merged

[ Translation ] Refine documents#1795
lvliang-intel merged 17 commits into
mainfrom
doc-translation

Conversation

@ZePan110
Copy link
Copy Markdown
Collaborator

@ZePan110 ZePan110 commented Apr 11, 2025

Description

[ Translation ] Refine documents

Issues

#1687

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)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

Copilot AI review requested due to automatic review settings April 11, 2025 06:26
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 11, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

Translation/docker_compose/intel/hpu/gaudi/README.md:43

  • The referenced script name (setup_env.sh) does not match the command later in the file (source ../set_env.sh); please standardize the script name for consistency.
To set up environment variables for deploying Translation services, source the _setup_env.sh_ script in this directory:

Translation/docker_compose/intel/cpu/xeon/README.md:43

  • The documentation references setup_env.sh while the command uses source ../set_env.sh; please align the script name to avoid user confusion.
To set up environment variables for deploying Translation services, source the _setup_env.sh_ script in this directory:

Comment thread Translation/README.md Outdated
Comment thread Translation/README.md Outdated
Comment thread Translation/README.md Outdated
Comment thread Translation/docker_compose/intel/cpu/xeon/README.md
Comment thread Translation/docker_compose/amd/gpu/rocm/README.md Outdated
Comment thread Translation/docker_compose/intel/hpu/gaudi/README.md Outdated
Comment thread Translation/docker_compose/amd/gpu/rocm/README.md Outdated
@joshuayao joshuayao added this to OPEA Apr 11, 2025
@joshuayao joshuayao added this to the v1.3 milestone Apr 11, 2025
@joshuayao joshuayao added the v1.3 label Apr 11, 2025
@joshuayao joshuayao moved this to In review in OPEA Apr 11, 2025
@joshuayao joshuayao added the documentation Improvements or additions to documentation label Apr 11, 2025
@joshuayao joshuayao linked an issue Apr 11, 2025 that may be closed by this pull request
1 task
ZePan110 and others added 4 commits April 11, 2025 19:47
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: ZePan110 <ze.pan@intel.com>
Comment thread Translation/README.md Outdated
Comment thread Translation/docker_compose/intel/cpu/xeon/README.md Outdated
Comment thread Translation/docker_compose/intel/cpu/xeon/README.md Outdated
Comment thread Translation/docker_compose/intel/cpu/xeon/README.md Outdated
Comment thread Translation/docker_compose/intel/hpu/gaudi/README.md Outdated
Comment thread Translation/docker_compose/intel/hpu/gaudi/README.md Outdated
Comment thread Translation/docker_compose/intel/hpu/gaudi/README.md Outdated
Comment thread Translation/README.md Outdated
Copy link
Copy Markdown
Collaborator

@yinghu5 yinghu5 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@lvliang-intel lvliang-intel merged commit 0967fca into main Apr 20, 2025
12 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in OPEA Apr 20, 2025
@lvliang-intel lvliang-intel deleted the doc-translation branch April 20, 2025 08:18
yongfengdu pushed a commit to yongfengdu/GenAIExamples that referenced this pull request May 8, 2025
Signed-off-by: ZePan110 <ze.pan@intel.com>
chyundunovDatamonsters pushed a commit to chyundunovDatamonsters/OPEA-GenAIExamples that referenced this pull request May 16, 2025
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
letonghan pushed a commit that referenced this pull request Sep 17, 2025
* Fix for react due to langgraph api change.

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

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

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ying Hu <ying.hu@intel.com>
cogniware-devops pushed a commit to Cogniware-Inc/GenAIExamples that referenced this pull request Dec 19, 2025
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: cogniware-devops <ambarish.desai@cogniware.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation v1.3

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Doc] Refine the README for Translation

5 participants