-
Notifications
You must be signed in to change notification settings - Fork 5.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
feat: Support for Autodetect in AstraDBVectorStore settings #4869
Conversation
CodSpeed Performance ReportMerging #4869 will degrade performances by 55.69%Comparing Summary
Benchmarks breakdown
|
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.
lgtm
Hey @erichare Can we merge this? |
@ogabrielluiz i'm a little cautious about it, mostly because i'm not completely positive how the dropdown selector for collections conflicts with the ASTRA_ENHANCED functionality in DS LF. Unless there's a really good reason i'd lean towards maybe waiting on this, and i'll try to rally some testers of all the scenarios and we can get it merged next week? i'll be out most of today and tomorrow as well |
…-ai#4869) * feat: first pass at autodetect updates * [autofix.ci] apply automated fixes * Fully support autodetect --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
This pull request [DO NOT MERGE] is a first pass at adding a much better user experience for existing collections, including
autodetect
support in AstraDBVectorStore