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/inject employee context to prompt #72

Merged
merged 2 commits into from
Nov 11, 2024

Conversation

onimsha
Copy link

@onimsha onimsha commented Nov 11, 2024

This pull request includes multiple changes to improve the functionality and efficiency of the chat and prompt systems. The most important changes include updates to the persona configurations, enhancements to the prompt system, and refactoring of imports in various files.

Persona Configuration Updates:

  • Increased the default number of chunks to include as context from 20 to 50 in personas.yaml.

Prompt System Enhancements:

  • Updated the system prompt to include specific information about Mindvalley’s operations and employee details in prompts.yaml.
  • Added new configurations for Airtable to retrieve Mindvalley employee information in app_configs.py.
  • Implemented a new function to add employee context to prompts, utilizing Redis for caching in prompt_utils.py.

Refactoring and Code Cleanup:

  • Refactored import statements for better readability and organization across multiple files, including process_message.py, answer.py, citations_prompt.py, and chat_backend.py. [1] [2] [3] [4]

User Email Integration:

  • Added user email handling in the Answer class and related functions to personalize responses based on user information in answer.py. [1] [2] [3]

These changes collectively enhance the chat system's ability to provide more personalized and contextually relevant responses while improving code maintainability.

Signed-off-by: Alex Co <alex.tuan@mindvalley.com>
Signed-off-by: Alex Co <alex.tuan@mindvalley.com>
@onimsha onimsha merged commit 10c8811 into main Nov 11, 2024
0 of 5 checks passed
@onimsha onimsha deleted the feat/inject-employee-context-to-prompt branch November 11, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant