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

Added llama unoptimized model code sample #35

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

Conversation

aidan-ricci
Copy link

Amazon Bedrock Agent Samples Pull Request

Instructions

  • Do not erase any parts of this template that are not applicable to your pull request.
  • If a section is not applicable, explicitly state the reason.
    • Tick the checkboxes for the items you have completed.
  • These are mandatory requirements, not mere suggestions.

Describe your changes

  • I added two files to the unoptimized model code sample.
  • agent_llama.py, a agents python that has updated methods for a cross-region inference profile
  • llama.ipynb, a notebook which creates the same restaurant agent as the mistral code sample, but with llama to show that it can be done with llama as well
Changes to ..., because ...

Issue ticket number and link

  • Issue # (if applicable)

All Submissions:

  • [y ] Have you followed the guidelines in our Contributing document?
  • [y ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • [n ] Are you uploading a dataset?
  • [ n] Have you added contributions to RELEASE NOTES?

New Example Submissions:

  • [ y] Have you tested your code, and made sure the functionality runs successfully? Provide screenshots.
  • [ y] Have you linted your Python code with black?
  • [ n] Does this implementation use the shared tools src/utils/*. List them here:
1. web_search
2. ...
  • [n ] Does this implementation use the helper functions src/utils/*. List them here:
1. bedrock_agent_helper.py
2. ...

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.

  • Business justification for including a new tool
This tool is necessary because ...
  • How is this tool implemented?
      • AWS CDK
      • AWS CloudFormation (recommended)

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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