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

feat: Multilingual Support and Code Quality Enhancements for v1 Release #48

Merged
merged 32 commits into from
Nov 29, 2023

Conversation

parambharat
Copy link
Contributor

Summary

This PR introduces key enhancements and fixes to wandbot, focusing on multilingual capabilities and code quality improvements.

Highlights

Multilingual Features: Added support for English and Japanese in chat bots; included language in database schema and routing.
Code Quality: Fixed issues related to timestamps, linting, and type annotations. Converted Slack app to async and improved docstrings and type annotations.
Configuration Updates: Resolved naming conflicts in configurations and updated environmental requirements.
Documentation: Updated README with new tokens and run commands.

Impact

These changes significantly enhance the bot's language handling capabilities and improve overall code maintainability and performance.

Bharat Ramanathan and others added 29 commits November 20, 2023 13:03

EN_INTRO_MESSAGE = (
"🤖 Hi {mention}: \n\n"
f"Please note that **wandbot is currently in alpha testing** and will experience frequent updates.\n\n"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Get rid of the alpha testing bit

LanguageFilterPostprocessor(languages=[language, "python"]),
CohereRerank(top_n=5, model="rerank-multilingual-v2.0")
if language == "ja"
else CohereRerank(top_n=5, model="rerank-english-v2.0"),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

reverse the condition and make language en

Copy link
Contributor Author

Choose a reason for hiding this comment

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

delete this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

add a readme and link to ayush's report.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ayulockin Please take a look here at the README.md

Copy link
Contributor

@ArtsiomWB ArtsiomWB left a comment

Choose a reason for hiding this comment

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

lgtm 🚢

@parambharat parambharat merged commit 983e522 into main Nov 29, 2023
4 checks passed
@parambharat parambharat deleted the fix/ja-responses branch December 18, 2023 06:39
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.

2 participants