-
Notifications
You must be signed in to change notification settings - Fork 62
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
Tuning bedrock_agents high-level classes to be simpler to use #20
base: main
Are you sure you want to change the base?
Conversation
… than all construction up front on creation). Added Tool2 (needs to be unified w/ Tool) and an alt version of portfolio demo.
…an to expand its scope), moved some kb attach code out of Agents.init(). Added some alt example versions for now.
…an to expand its scope), moved some kb attach code out of Agents.init(). Added some alt example versions for now.
…hared tools for use by multiple agents (ex. web_search), simplifying creation syntax
… code in consumers.
…e agents_helper. Removed direct references to agents_helper from examples.
…ase_helper.py to retrieve s3 bucket name if needed.
…ability to use local Python functions as Tools (but it materializes them to a lambda for now)
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Issue #, if available: Description of changes: By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. |
Can you complete the above checklist ^ ? |
…y examples/bedrock-agent to examples/bedrock_agent to match library name
Extra text for answers: Does this implementation use the shared tools src/utils/*. List them here:
Does this implementation use the helper functions src/utils/*. List them here:
Have you included your Agent Example name and introduction in README.md src/utils Submissions: src/shared tool Submissions: Business justification for including a new tool I added a command-line utility, src/shared/tool_manager to allow testing the Toolbox (tool sharing via a registry for simpler creation). I also added a note in shared/web_search/README.md explaining how to register web_search and sample code in examples/bedrock_agent/getting_started.ipynb that uses it to attach the web_search tool to an agent (see "Working with Tools", "Attaching tools from the toolbox". The tool_manager does not use CDK or CloudFormation, it calls the Toolbox class to register the Tool's metadata into a DynamoDB table that it creates. If we wanted to instead do that in the CDK for the shared tools, we could do that. |
Amazon Bedrock Agent Samples Pull Request
Instructions
Describe your changes
Issue ticket number and link
All Submissions:
New Example Submissions:
src/utils/*
. List them here:src/utils/*
. List them here:src/utils Submissions:
Changes to the utils folder won't be accepted. Instead, open a new issue.
src/shared tool Submissions:
Changes to existing tools won't be accepted. Instead, open a new issue.