fix: Set default search_type to 'similarity' in as_retriever method of AzureSearch #28376
Conversation
Adding Vector Store SQL DB
Adding SQLServer Vector Store to the Index Table
…m90/langchain into sqlvector-langchain-branch
adding sqlserver vectore store matrix
…m90/langchain into sqlvector-langchain-branch
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
efriis
left a comment
efriis
left a comment
There was a problem hiding this comment.
I believe the search_type must be a string, and it defaults to "hybrid", so I don't think this PR makes sense
|
i.e. if you look at the default values in the |
|
Thank you for your comment, @efriis . The intent of this PR is to align with the comment and documentation for As you've correctly pointed out, the default value of |
5 participants
Description
This PR updates the
as_retrievermethod in theAzureSearchto ensure that thesearch_typeparameter defaults to 'similarity' when not explicitly provided.Previously, if the
search_typewas omitted, it did not default to any specific value. So it was inherited fromAzureSearchVectorStoreRetriever, which defaults to 'hybrid'.This change ensures that the intended default behavior aligns with the expected usage.
Issue
No specific issue was found related to this change.
Dependencies
No new dependencies are introduced with this change.