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

fix: simple_summary_wrapper function_call KeyError #1265

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

djkazic
Copy link

@djkazic djkazic commented Apr 16, 2024

Fixes #1261

How to test
Use ollama and the default dolphin model and spam the chat until it hits the context window limit and an automatic /summarize is triggered. Alternatively, just use /summarize.

Have you tested this PR?
Yes

Is your PR over 500 lines of code?
No

@djkazic djkazic changed the title Fix simple_summary_wrapper function_call KeyError fix: simple_summary_wrapper function_call KeyError Apr 16, 2024
@madgrizzle
Copy link
Contributor

Is there a way to update this so that checks are successful? This fix works for me and summarize is otherwise broken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To triage
Development

Successfully merging this pull request may close these issues.

/summarize command fails
2 participants