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

Update _index.md #27115

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update _index.md #27115

wants to merge 2 commits into from

Conversation

barieom
Copy link
Contributor

@barieom barieom commented Jan 14, 2025

What does this PR do? What is the motivation?

Merge instructions

Merge readiness:

  • Ready for merge

Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the <yourname>/description naming convention) and then add the following PR comment:

/merge

Additional notes

@barieom barieom requested a review from a team as a code owner January 14, 2025 20:39
@barieom barieom closed this Jan 14, 2025
@barieom barieom reopened this Jan 14, 2025
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

Copy link
Contributor

@janine-c janine-c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for resubmitting with a new branch name! I added a suggestion to make it blend in a bit better with the rest of the docs, but chose to make it optional, since I know legalese can be tough to edit.

@@ -37,10 +37,12 @@ Select a topic cluster from the list to examine how inputs or outputs about spec

{{< img src="llm_observability/cluster_map/box.png" alt="The box packing layout displays clusters of traces represented by colored circles, and includes a panel listing clusters with topics, trace counts, and failure rates." style="width:100%;" >}}

<div class="alert alert-info">Cluster Map feature includes components built with Llama</div>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<div class="alert alert-info">Cluster Map feature includes components built with Llama</div>
<div class="alert alert-info">The Cluster Map feature includes components built with Llama.</div>

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.

2 participants