-
Notifications
You must be signed in to change notification settings - Fork 44.6k
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
Bug Fix: summarize_text function usage #4863
Conversation
✅ Deploy Preview for auto-gpt-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #4863 +/- ##
=======================================
Coverage 50.03% 50.03%
=======================================
Files 116 116
Lines 4831 4831
Branches 650 650
=======================================
Hits 2417 2417
Misses 2229 2229
Partials 185 185
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically a duplicate of #4876, but more complete and raised to master
this time. LGTM!
* Fix summarize_text function usage * Update memory_item.py
Background
Incorrect usage of
summarize_text
function within recursionChanges
Update the parameters passed to
summarize_text
function i.e. add config paramDocumentation
Not required
Test Plan
No changes
PR Quality Checklist