-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Demo UI add env vars & other small fixes #1828
Conversation
ZanSara
commented
Nov 30, 2021
•
edited
Loading
edited
- Add the possibility to set more value through env vars
- Add some more questions for the Random Questions list (tested)
- Refine some error messages and add logs
- Align docker-compose.yml and docker-compose-gpu.yml on the exposed env vars and fix a few typos
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.
LGTM. Feel free to merge after addressing the comment below
docker-compose-gpu.yml
Outdated
@@ -52,5 +52,8 @@ services: | |||
environment: | |||
- API_ENDPOINT=http://haystack-api:8000 | |||
- EVAL_FILE=eval_labels_example.csv | |||
- HAYSTACK_UI_DISABLE_FILE_UPLOAD # docker-compose run -e HAYSTACK_UI_DISABLE_FILE_UPLOAD=1 ... | |||
- DISABLE_FILE_UPLOAD # DISABLE_FILE_UPLOAD=1 docker-compose up |
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.
don't get the comment 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.
Is docker-compose in this example taking the value for the env variable from the host machine? If yes, let's clarify this in a comment for all the next variables
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.
Yeah looking at it now it is way more cryptic than it should 😄
…r to prevent spamming
…r to prevent spamming
… into demo_ui_add_env_vars