-
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
Drop AbstractSingleton as a parent class of the vector memory #4901
Drop AbstractSingleton as a parent class of the vector memory #4901
Conversation
✅ Deploy Preview for auto-gpt-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #4901 +/- ##
==========================================
- Coverage 50.29% 50.20% -0.10%
==========================================
Files 116 116
Lines 4843 4808 -35
Branches 653 645 -8
==========================================
- Hits 2436 2414 -22
+ Misses 2221 2215 -6
+ Partials 186 179 -7
☔ View full report in Codecov by Sentry. |
You changed AutoGPT's behaviour. The cassettes have been updated and will be merged to the submodule when this Pull Request gets merged. |
…icant-Gravitas#4901) * Drop AbstractSingleton as a parent class of the vector memory * Forgot pytest fixture in integration tests * Revert typo
…icant-Gravitas#4901) * Drop AbstractSingleton as a parent class of the vector memory * Forgot pytest fixture in integration tests * Revert typo
Background
Singletons are bad and we don't need them
Changes
Documentation
N/A
Test Plan
CI
PR Quality Checklist