-
Notifications
You must be signed in to change notification settings - Fork 337
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
Add Amazon Bedrock Knowledge base as a RAG Engine (retriever) #427
Conversation
…nto feat_bedrock_kb
…nto feat_bedrock_kb
…nto feat_bedrock_kb
…nto feat_bedrock_kb
can I build the chatbot off of this branch? |
…nto feat_bedrock_kb
…nto feat_bedrock_kb
…m-chatbot into feat_bedrock_kb
Yes, you can now use this branch to deploy as it has been synched with |
@massi-ang Thank you so much, this worked for me |
There are few conflicts that needs to be resolved first |
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.
Great feature to add.
Added minor comments. Will approved after dev files are removed.
enterprise: false | ||
enterprise: false, | ||
}, | ||
knowledgeBase: { |
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.
nit: suggestion bedrock or bedrockKnowledgeBase. Other properties name the service.
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.
Do they? I see aurora
, kendra
, ...
lib/shared/layers/python-sdk/python/genai_core/bedrock_kb/client.py
Outdated
Show resolved
Hide resolved
lib/user-interface/react-app/src/pages/rag/create-workspace/create-workspace.tsx
Show resolved
Hide resolved
lib/user-interface/react-app/src/pages/rag/create-workspace/kb-form.tsx
Outdated
Show resolved
Hide resolved
lib/user-interface/react-app/src/pages/rag/create-workspace/kb-form.tsx
Outdated
Show resolved
Hide resolved
…m-chatbot into feat_bedrock_kb
- remove dev files - remove console.log - remove unused code
…nto feat_bedrock_kb
…mples#427) * feat(kb): initial * feat(bedrock_kb): initial * feat(bedrock_kb): knowledge base support * feat(bedrock_kb): delete workspaces * feat(bedrock_kb): config * feat(bedrock_kb): hybrid search * feat(bedrock_kb): upgrade boto3 * fix(cohere_embeddings): correct set the `input_type` * feat(kb): add Bedrock KB to the welcome page * fix(no unused var): correct ignore * fix: correct enum values * feat(bedrock_kb): metadata filters * feat(bedrock_kb): dedup * feat(llama3): adapter * feat(bedrock_kb): merge to latest * chore: updating code to use langchain-community * feat(bedrock): fix error * fix(bedrock_kb): magic cli * fix: correct path to compiled files * chore: removed wrongly tracked file * fix: fix review feedback - remove dev files - remove console.log - remove unused code * fix: remove feature under development * fix: typescript errors * fix: ignore code un `dist` to avoid recursive builds * tests: update test snapshot and fix call order * test: fix rag tests
Issue #, if available:
Resolves #271
Description of changes:
With this PR we are adding Amazon Bedrock Knowledge Bases as additional RAG Engine (retrievers).
The KB must be already created and available in the same region used by Bedrock. The integration supports both semantic retrieval and hybrid retrieval via the KB retrieve API.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.