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

react-ui: Add support to display Chinese #713

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

bjzhjing
Copy link
Contributor

@bjzhjing bjzhjing commented Sep 2, 2024

llm-tgi microservice from GenAIComps has encoded each output text, so Chinese response will be shown as hexadecimal in react UI. Add support to decode and display the response in Chinese correctly.

@bjzhjing
Copy link
Contributor Author

bjzhjing commented Sep 3, 2024

@chensuyue @lvliang-intel Hi Suyue, Liang, I see the following errors from CI run-test:

Error response from daemon: Conflict. The container name "/retriever-redis-server" is already in use by container "dbb34303fb2920533db073b87dad6adbc3fa935471f575661037fdf2ce1e4e88". You have to remove (or rename) that container to be able to reuse that name.
Error: Process completed with exit code 1.

But I don't think it's related to the code I changed. Could you please help take a look and let me know if anything I missed.

bjzhjing and others added 2 commits September 3, 2024 13:51
llm-tgi microservice from GenAIComps has encoded each text, so Chinese
response will be shown as hexadecimal in react UI. Add support to decode
and display the response in Chinese. Also return raw response if no
pattern found.

Signed-off-by: Cathy Zhang <cathy.zhang@intel.com>
Signed-off-by: Ruoyu Ying <ruoyu.ying@intel.com>
@lvliang-intel lvliang-intel merged commit 8c40204 into opea-project:main Sep 3, 2024
14 checks passed
xuechendi pushed a commit to xuechendi/GenAIExamples that referenced this pull request Sep 9, 2024
* react-ui: Add support to display Chinese

llm-tgi microservice from GenAIComps has encoded each text, so Chinese
response will be shown as hexadecimal in react UI. Add support to decode
and display the response in Chinese. Also return raw response if no
pattern found.

Signed-off-by: Cathy Zhang <cathy.zhang@intel.com>
Signed-off-by: Ruoyu Ying <ruoyu.ying@intel.com>

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

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

---------

Signed-off-by: Cathy Zhang <cathy.zhang@intel.com>
Signed-off-by: Ruoyu Ying <ruoyu.ying@intel.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
dmsuehir pushed a commit to dmsuehir/GenAIExamples that referenced this pull request Sep 11, 2024
* react-ui: Add support to display Chinese

llm-tgi microservice from GenAIComps has encoded each text, so Chinese
response will be shown as hexadecimal in react UI. Add support to decode
and display the response in Chinese. Also return raw response if no
pattern found.

Signed-off-by: Cathy Zhang <cathy.zhang@intel.com>
Signed-off-by: Ruoyu Ying <ruoyu.ying@intel.com>

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

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

---------

Signed-off-by: Cathy Zhang <cathy.zhang@intel.com>
Signed-off-by: Ruoyu Ying <ruoyu.ying@intel.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

3 participants