-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat(llm-observability): trace scene #27562
Conversation
b327c42
to
2e24346
Compare
Size Change: +83 B (+0.01%) Total Size: 1.13 MB ℹ️ View Unchanged
|
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.
Pushed a couple commits to be able to test this better locally - most importantly self-capture of Max AI generation.
Looking good, just a few polishes I think are worth adding!
frontend/src/scenes/llm-observability/LLMObservabilityTraceScene.tsx
Outdated
Show resolved
Hide resolved
frontend/src/scenes/llm-observability/LLMObservabilityTraceScene.tsx
Outdated
Show resolved
Hide resolved
frontend/src/scenes/llm-observability/LLMObservabilityTraceScene.tsx
Outdated
Show resolved
Hide resolved
frontend/src/scenes/llm-observability/llmObservabilityTraceLogic.ts
Outdated
Show resolved
Hide resolved
import { llmObservabilityLogic } from './llmObservabilityLogic' | ||
import type { llmObservabilityTraceLogicType } from './llmObservabilityTraceLogicType' | ||
|
||
export interface LLMObservabilityTraceDataNodeLogicParams { |
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.
That is a really verbose name 🙈
frontend/src/scenes/llm-observability/LLMObservabilityTracesScene.tsx
Outdated
Show resolved
Hide resolved
@Twixes updated screenshots: |
Co-authored-by: Michael Matloka <[email protected]>
Problem
We want to have a view where generations are grouped in a single sorted list to debug the agent.
Changes
This PR adds a basic scene.
Demo
Does this work well for both Cloud and self-hosted?
Yes
How did you test this code?
Manual testing + backend unit tests.