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.

Sign up for GitHub

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

Jump to bottom

DocSum set_env.sh has wrong parameter TGI_LLM_ENDPOINT #670

Closed
mandalrajiv opened this issue Aug 27, 2024 · 0 comments · Fixed by #676
Closed

DocSum set_env.sh has wrong parameter TGI_LLM_ENDPOINT #670

mandalrajiv opened this issue Aug 27, 2024 · 0 comments · Fixed by #676
Assignees
@yinghu5
Labels
aitce

Comments

@mandalrajiv
Copy link
Contributor

Refer to the GenAI Example DocSum set_env.sh file - https://github.com/opea-project/GenAIExamples/blob/main/DocSum/docker/set_env.sh.

It has a line like below.

export TGI_LLM_ENDPOINT="http://${your_ip}:8008"

It should be:
export TGI_LLM_ENDPOINT="http://${host_ip}:8008"

Since your_ip is not defined elsewhere, the document sumamrization tasks runs into an invalid url issue as seen in the docker logs. In the readme, we define the value of host_ip environment variable. So host_ip should be used instead of your_ip.
yinghu5 added a commit that referenced this issue Aug 28, 2024
@yinghu5
Update set_env.sh
71fbceb 
to fix #670
@yinghu5 yinghu5 mentioned this issue Aug 28, 2024
Merged
4 tasks
dmsuehir pushed a commit to dmsuehir/GenAIExamples that referenced this issue Sep 11, 2024
@yinghu5 @dmsuehir
Yinghu5 patch 1 (opea-project#676)
84903d0 
* Update set_env.sh

to fix opea-project#670

* Update README.md

fixed your_ip and host_ip

* Update README.md

fix your_ip and host_ip
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yinghu5 yinghu5

Labels
aitce
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Yinghu5 patch 1 opea-project/GenAIExamples
3 participants
@preethivenkatesh @yinghu5 @mandalrajiv