Skip to content
This repository has been archived by the owner on Dec 11, 2024. It is now read-only.

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.

@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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant