-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Expanded Support for Vectara's Search API #2206
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
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -212,6 +254,62 @@ export class VectaraStore extends VectorStore { | |||
return documentsAndScores; |
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.
This PR adds a new method similaritySearchWithSummary
that makes an external HTTP request using fetch
to the Vectara API endpoint. This comment is flagging the change for maintainers to review the addition of the new request.
This is great - do you think this could make sense as a new https://github.com/hwchase17/langchainjs/pull/2170/files That way, you could initialize a retriever in this mode and it would automatically call If the summary functionality is also possible with specific args to |
Thanks for suggesting this! I'll take a look at this approach soon. I definitely didn't feel great about adding an entire method that will throw an error by default for most VectorStore integrations. I'll see about making this better. Might ping you with some questions. |
+1 for a new search type. We'll be adding this to Zep soon. |
+1 from me as well
Happy to help with this PR too if needed. Pls lmk.
…On Tue, Aug 15, 2023 at 4:27 PM Daniel Chalef ***@***.***> wrote:
+1 for a new search type. We'll be adding this to Zep soon.
—
Reply to this email directly, view it on GitHub
<#2206 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAN5GO7BLRJZE6KLOOTFWQTXVQAV3ANCNFSM6AAAAAA3IIYHCI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
5a94f4b
to
c7d157d
Compare
Hey @Kae7in is this ready for review? |
Hey @Kae7in , @jacoblee93 - any updates on this? |
Update? |
Add support for Context Config, Generative Summarization, and more
This PR gives users of LangChainJS's Vectara integration more fine-grained control over Vectara's Search API.
New
similaritySearchWithSummary()
method allows for use of Vectara's Generative Summarization feature.New optional fields on
similaritySearch()
methods:contextConfig
exposes additional query context properties that the system may use to refine the results.start
field controls the starting position within the list of results.rerankingConfig
enables reranking of results, to further increase relevance in certain scenarios.summary
field allows you to use Vectara's "Gounded Generation", a groundbreaking way of producing generative summaries on top of your own data.Other additions:
similaritySearchWithSummary(...)
method has been added to the baseVectorStore
class, which also paves the way for other Vector databases that might implement similar summarization techniques in the near future.About me:
@KaelinHooper