-
Notifications
You must be signed in to change notification settings - Fork 2
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
[CAI-198] - Chatbot Languages #1206
Conversation
|
apps/chatbot/src/modules/chatbot.py
Outdated
ITALIAN_THRESHOLD = 0.85 | ||
NUM_MIN_WORDS_QUERY = 3 | ||
NUM_MIN_REFERENCES = 1 | ||
USE_PRESIDIO = True if os.getenv("CHB_USE_PRESIDIO", "True") == "True" else False |
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.
To avoid issue with capital letters I would lower the env variable
USE_PRESIDIO = True if os.getenv("CHB_USE_PRESIDIO", "True") == "True" else False | |
USE_PRESIDIO = True if lower(os.getenv("CHB_USE_PRESIDIO", "True")) == "true" else False |
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.
Done here and in src/modules/engine.py
.
the method lower()
goes after the string. lower(string)
does not work; string.lower()
works.
Jira Pull Request LinkThis Pull Request refers to the following Jira issue CAI-198 |
List of Changes
Motivation and Context
privacy and UI reasons
How Has This Been Tested?
jupyter notebook
Screenshots (if appropriate):
Types of changes
Checklist: