-
Notifications
You must be signed in to change notification settings - Fork 130
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
deprecate: Added deprecations for methods superseded by query_points #843
Conversation
✅ Deploy Preview for poetic-froyo-8baba7 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
qdrant_client/qdrant_client.py
Outdated
warnings.warn( | ||
"`search_matrix_pairs` method is deprecated and will be removed in the future." | ||
" Use `query_points` instead.", | ||
DeprecationWarning, | ||
stacklevel=2, | ||
) |
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.
we've just introduced it, we don't want to deprecate it yet :')
qdrant_client/qdrant_client.py
Outdated
warnings.warn( | ||
"`search_matrix_offsets` method is deprecated and will be removed in the future." | ||
" Use `query_points` instead.", | ||
DeprecationWarning, | ||
stacklevel=2, | ||
) |
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.
same as with search_matrix_pairs
…843) * deprecate: Added deprecations for methods superseded by query_points * deprecate: Undeprecate search_matrix_offsets and search_matrix_pairs
Show deprecation message to methods that are suppressed by query_points (recommend, discover, search, context search)
All Submissions:
dev
branch. Did you create your branch fromdev
?New Feature Submissions:
pre-commit
withpip3 install pre-commit
and set up hooks withpre-commit install
?Changes to Core Features: