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
Amazon Bedrock agent example #135
Conversation
Hey @begimher , thanks for the contribution. I noticed there is a duplicated code in the file, is this expected? |
examples/amazon_bedrock.py
Outdated
agentId=agent_id, | ||
agentAliasId=agent_alias_id, | ||
sessionId=session_id, | ||
inputText=prompt, |
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.
Should this be sanitized_prompt?
examples/amazon_bedrock.py
Outdated
region = "" | ||
agent_id = "" | ||
agent_alias_id = "" | ||
session_id = "" |
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.
Since these are constants they should be capitalized.
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.
Ruff is failing, have you installed pre-commit?
examples/amazon_bedrock.py
Outdated
input_scanners = [Anonymize(vault), Toxicity(), TokenLimit(), PromptInjection()] | ||
output_scanners = [Deanonymize(vault), NoRefusal(), Relevance(), Sensitive()] | ||
|
||
prompt = ( |
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.
Capitlization?
Thank you @CandiedCode @asofter ! |
@begimher thank you for the contribution |
Change Description
An example code to invoke Amazon Bedrock agent
Issue reference
N/A
Checklist