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

Auto summarize conversations, add attachment count #1341

Merged
merged 3 commits into from
Dec 23, 2024
Merged

Conversation

Josh-XT
Copy link
Owner

@Josh-XT Josh-XT commented Dec 23, 2024

This pull request introduces enhancements to the conversation management and database schema in the agixt project. The key changes include adding support for conversation summaries and attachment counts, updating the database schema to include these new fields, and ensuring the conversation summary is updated during interactions.

Enhancements to conversation management:

  • agixt/Conversations.py: Added methods set_conversation_summary, get_conversation_summary, get_attachment_count, update_attachment_count, and increment_attachment_count to manage conversation summaries and attachment counts.
  • agixt/Conversations.py: Updated get_conversations_with_detail to include summary and attachment_count in the returned conversation details.

Database schema updates:

  • agixt/DB.py: Added summary and attachment_count columns to the Conversation class.
  • agixt/DB.py: Implemented ensure_conversation_summary function to add the new columns to the database schema if they do not already exist. [1] [2]

Conversation summary updates:

  • agixt/XT.py: Incremented attachment count when a file is downloaded in learn_from_file.
  • agixt/XT.py: Updated the conversation summary after a chat completion in chat_completions.

New prompt template:

@Josh-XT Josh-XT marked this pull request as ready for review December 23, 2024 20:39
@Josh-XT Josh-XT merged commit a2d75e3 into main Dec 23, 2024
7 checks passed
@Josh-XT Josh-XT deleted the summarize-convo branch December 23, 2024 20:46
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.

1 participant