- Notifications
You must be signed in to change notification settings - Fork 514
Add Amazon Bedrock Guardrails Improvements #12422
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 Guardrails Improvements #12422
Conversation
🚀 Benchmarks reportTo see the full report comment with |
| Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
muthu-mps left a comment
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.
LGTM!
| Are we using LLM or models in the charts ? cc: @daniela-elastic |
@daniela-elastic , the alternate option is to use "by foundation models". Reference. What do you suggest? |
| Yes, dashboard looks LGTM. Yes, let's make the titles FM or models instead of LLM. Rest, looks good. |
💚 Build Succeeded
History
cc @agithomas |
|
| Package aws_bedrock - 0.22.0 containing this change is available at https://epr.elastic.co/package/aws_bedrock/0.22.0/ |
| @agithomas We were reviewing the GCP vertex AI dashboard. Some feedback from AWS as part of those discussions.
cc: @daniela-elastic |
| For no1 yes it makes sense as long as we don't change the name of the widget - "umber of invocations where the guardrails intervened" |
| I'm fine to call it "- by foundation model" (singular, not plural). Or even just "by model". Is there any reason why we need to specify that it's "foundation" or it can just be assumed? |
I think model should be a generic term that we can use across LLM Integrations. |
The term "Foundation Models" originated in a 2021 paper (https://arxiv.org/pdf/2108.07258). While almost all models we are plotting here qualify as Foundation Models (FM), except most embedding models, because:
But yes in case an embedding model starts showing these properties, then they come under FM. Since "models" is a superset of "foundation models", using "model" as also suggested by Daniela seems a better choice for the dashboard - it looks cleaner while future-proofing our terminology for any new model types that may not exhibit foundation model properties. |
| @agithomas - The Without hover Hover over the total invocations chart. |
| Presently, the amazon bedrock integration has 8.16.2 dependency. I think it may not be good to increment the version to |
* Add Amazon Bedrock Guardrails Improvements
* Add Amazon Bedrock Guardrails Improvements






Proposed commit message
Add improvements to the Bedrock overview dashboard and Guardrails dashboard
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots
Amazon Bedrock Overview dashboard
Guardrails Overview Dashboard
