Conversation
🦋 Changeset detectedLatest commit: b47cf8c The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
* save query and evaluate functions refactoring * dynamodb query put * presidio * changeset * Update chatbot app queries * Update chatbot-monitor scripts * Update chatbot-monitor scripts * Update chatbot queries.py --------- Co-authored-by: mdciri <marco.cirillo@uqido.com> Co-authored-by: Marco Domenico Cirillo <59966344+mdciri@users.noreply.github.com>
|
This pull request is stale because it has been open for 14 days with no activity. If the pull request is still valid, please update it within 21 days to keep it open or merge it, otherwise it will be closed automatically. |
* save query and evaluate functions refactoring * dynamodb query put * presidio * changeset * Update chatbot app queries * Update chatbot-evaluator modules * Update poetry files * Update lambda function * Update chatbot monitor scripts * Update chatbot-evaluate scripts * Update chatbot-evaluate lamda function * Update chatbot-monitor lambda function * Add changeset * Update chatbot-monitor monitor.py * Update chatbot-monitor monitor.py * Align to CAI-629 * Update lambda function to consider only unique records * Update lambda function in chatbot-monitor * Update chatbot-evaluate lambda function * Update chatbot-evaluate tests * Update loggers in chatbot monitor, index, and evaluate * Update loggers in chatbot, monitor, index, and evaluate * Update dockerfile for tests * Add new changeset * fix: logger param * fix: comment langfuse add score. We will use SQS queue in another pr * comment calls to presidio. In a future pr we will use SQS for them --------- Co-authored-by: batdevis <devis.battisti@uqido.com>
|
This pull request is stale because it has been open for 14 days with no activity. If the pull request is still valid, please update it within 21 days to keep it open or merge it, otherwise it will be closed automatically. |
* run Docker as normal user * user home * changeset * user home Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: marcobottaro <39835990+marcobottaro@users.noreply.github.com>
Jira Pull Request LinkThis Pull Request refers to the following Jira issue CAI-629 |
Branch is not up to date with base branch@marcobottaro it seems this Pull Request is not updated with base branch. |
|
This PR exceeds the recommended size of 800 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size. |
List of Changes
This pull request introduces major architectural changes to the chatbot system, focusing on refactoring, modularization, and improved payload handling between components. Key improvements include moving monitoring-related functionality from
chatbottochatbot-monitor, introducing payload compression for SQS communication, and updating dependencies for better compatibility and performance.Architectural Refactoring and Modularization:
chatbottochatbot-monitorto better separate monitoring and evaluation concerns.chatbot-evaluateto send score writing operations tochatbot-monitorinstead of directly handling them, removing the dependency on Langfuse client code inchatbot-evaluate.Payload Compression and SQS Communication:
codec.pyand updatedlambda_function.pyto compress payloads before sending them to SQS, improving efficiency and reliability of inter-service communication.Dependency and Environment Updates:
pyproject.tomlforllama-indexand related packages to newer versions, and addedjupyterfor development.git, ensuring build compatibility.Logging and Monitoring Enhancements:
Documentation and Miscellaneous:
README.mdfor better developer onboarding and system understanding.Motivation and Context
move presidio masking into the
chatbot-monitorlambda to speed-up thechatbotone. As result, the user should get the chatbot generated response in less time.How Has This Been Tested?
Locally using docker compose in
apps/chatbot/dockerand in the DEV environment.Screenshots (if appropriate):
Types of changes
Checklist: