Skip to content
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

Data Analyst Assistant for Video Games Sales #17

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

aurbac
Copy link

@aurbac aurbac commented Jan 17, 2025

Amazon Bedrock Agent Samples Pull Request

Describe your changes

Changes to ..., because ... 

Issue ticket number and link

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Example Submissions:

  • Have you tested your code, and made sure the functionality runs successfully?
  • Have you linted your python code with black?
  • List the shared tools that your example is using:
1. web_search
2. ...
  • My implementation uses helper functions in src/utils/*. List them here:
1. bedrock_agent_helper.py
2. ...
  • Have you included your Agent Example name in README.md?
  • Have you included your Agent Example introduction in README.md
  • Have you included Introduction, Architecture Diagram, Prerequisites, Usage, Sample Prompts, and Clean Up steps?
  • Have you appened _agent to the new folder name in src/examples?
  • I agree to resolve any issues created for this example in the future.

src/utils Submissions:

Changes to utils folder won't be accepted. Instead, open a new issue.

New src/shared tool Submissions:

  • Business justification for including a new tool
This tool is necessary because ...
  • How is this tool implemented?

    • AWS CDK
    • AWS CloudFormation (recommended)
  • I agree to resolve any issues created for this tool in the future.

Changes to src/shared tools:

Changes to existing tools won't be accepted. Instead, open a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant