Skip to content

AGENT-DRAFT: GKO-2024 — LLM Analytics & Dashboard Enhancements [code+dev-draft]#1319

Closed
Lyd1aCla1r3 wants to merge 2 commits intomainfrom
doc-update/gko-2024-1774367671
Closed

AGENT-DRAFT: GKO-2024 — LLM Analytics & Dashboard Enhancements [code+dev-draft]#1319
Lyd1aCla1r3 wants to merge 2 commits intomainfrom
doc-update/gko-2024-1774367671

Conversation

@Lyd1aCla1r3
Copy link
Copy Markdown
Contributor

Automated update: gko-2024

Product: gko
Version: 4.11

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!


### Elasticsearch Aggregation Scripts

The gateway uses Painless scripts to compute LLM metrics from additional-metrics fields. These scripts handle missing fields gracefully (defaulting to 0).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[vale] reported by reviewdog 🐶
Microsoft.Adverbs:Remove 'gracefully' if it's not important to the meaning of the statement. (https://docs.microsoft.com/en-us/style-guide/word-choice/use-simple-words-concise-sentences)


- LLM metrics require Elasticsearch-backed analytics (not available with in-memory or other analytics backends)
- `LLM_PROMPT_TOTAL_TOKEN` and `LLM_PROMPT_TOKEN_COST` only populate when LLM proxy policies write to `additional-metrics` fields
- Facet requests limited to 3 dimensions (`by` maxItems: 3)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[vale] reported by reviewdog 🐶
docs.Spelling:Did you really mean 'maxItems'?

- LLM metrics require Elasticsearch-backed analytics (not available with in-memory or other analytics backends)
- `LLM_PROMPT_TOTAL_TOKEN` and `LLM_PROMPT_TOKEN_COST` only populate when LLM proxy policies write to `additional-metrics` fields
- Facet requests limited to 3 dimensions (`by` maxItems: 3)
- Time-series requests limited to 2 grouping dimensions (`by` maxItems: 2)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[vale] reported by reviewdog 🐶
docs.Spelling:Did you really mean 'maxItems'?

- Added dashboard creation steps (navigate, create from template, select LLM, adjust filters)
- Added verification section
- Added common pitfalls warning (backend not started, no data)
- Added next steps
- Fixed passive voice in step 4
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