-
Notifications
You must be signed in to change notification settings - Fork 183
Add Amazon Bedrock AgentCore to the LLM Observability page #4141
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
Conversation
Vale Linting ResultsSummary: 3 suggestions found 💡 Suggestions (3)
|
🔍 Preview links for changed docs |
daniela-elastic
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.
- Curious why we are putting Amazon Bedrock AgentCore before Amazon Bedrock. I would have reversed the order since AgentCore is part of Bedrock.
- Can we put Azure AI Foundry below Bedrock.
- Arrange order to be:
Amazon Bedrock
Amazon Bedrock AgentCore
Azure AI Foundry
Azure OpenAI
GCP VertexAI
OpenAI
Feel free to address, not need for further approvals
daniela-elastic
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.
Pls fix Agent -> Agentic (see comments). No need for further approvals
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.
We don't need to capitalize all of these terms if we refer to the branding guidelines - and we should try to keep that first H2 title more specific
Otherwise LGTM!
for reference: https://brand.elastic.co/302f66895/p/194a3b-writing-style-guide/t/19954f
Co-authored-by: florent-leborgne <[email protected]>
Co-authored-by: florent-leborgne <[email protected]>
Co-authored-by: florent-leborgne <[email protected]>
Co-authored-by: florent-leborgne <[email protected]>
Co-authored-by: florent-leborgne <[email protected]>
florent-leborgne
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
Summary
This PR adds Amazon Bedrock AgentCore to the LLM Observability page and renames various titles.
Fixes #4074
Generative AI disclosure