-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
bedrock[minor]: Allow knowledge bases to use filters and searchType #6189
bedrock[minor]: Allow knowledge bases to use filters and searchType #6189
Conversation
…eSearchType, Update the KB to support other locations as sources
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -39,15 +39,15 @@ | |||
"author": "LangChain", |
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.
Hey there! 👋 I noticed that this PR includes a change in the dependencies, specifically the update of "@aws-sdk/client-bedrock-agent-runtime" and "@aws-sdk/types". This is flagged for maintainers to review as it affects the peer/dev dependencies of the project. Keep up the great work! 🚀
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, thank you for pushing this improvement!
Thanks for the review and merge @bracesproul. Is it possible to bump the aws package? or there's a release process/waiting? |
Yes! Just cut a release. Should be avail in v |
Adds optional
filter
and optionaloverrideSearchType
parameters to the Bedrock knowledge base client:Also, updates the source location of the document depending on the KB configuration: S3, WEB, CONFLUENCE, SALESFORCE, SHAREPOINT