Skip to content

docs: replace evaluation loop image with LoopDiagram component#3237

Merged
Lotte-Verheyden merged 1 commit into
mainfrom
replace-eval-loop-image
Jul 4, 2026
Merged

docs: replace evaluation loop image with LoopDiagram component#3237
Lotte-Verheyden merged 1 commit into
mainfrom
replace-eval-loop-image

Conversation

@Lotte-Verheyden

@Lotte-Verheyden Lotte-Verheyden commented Jul 4, 2026

Copy link
Copy Markdown
Member

Replaces the static continuous-evaluation-loop image on the evaluation core concepts page with the interactive <LoopDiagram /> component used in the academy. This gives the docs page the same loop graph, with Online/Offline labels, per-stage meta tags, cycling activity indicator, and links into the academy stages.

Greptile Summary

Replaces the static continuous-evaluation-loop.png image (wrapped in a <Frame>) with the interactive <LoopDiagram /> component on the evaluation core-concepts page.

  • LoopDiagram is already registered globally in mdx-components.tsx, so no import is needed in the MDX file — the component will resolve correctly.
  • The component renders an animated, responsive loop diagram with Online/Offline labels, per-stage meta tags, a cycling activity indicator, and links to the corresponding academy stages.

Confidence Score: 5/5

Safe to merge — single-line docs change swapping a static image for an already-registered interactive component.

The only change is removing 4 lines (a <Frame> wrapping a PNG) and adding <LoopDiagram />. The component is globally registered in mdx-components.tsx so it resolves without an explicit import, it is already live on overview.mdx and multiple academy pages, and no logic or configuration is affected.

No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
    subgraph Online["Online Evaluation"]
        T["Trace\n(traces · sessions · agents · prompts)"]
        M["Monitor\n(dashboards · LLM-as-judge · feedback)"]
    end
    subgraph Offline["Offline Evaluation"]
        D["Build Datasets\n(datasets · features-as-tests)"]
        E["Experiment\n(prompts · models · code variants)"]
        EV["Evaluate\n(judges · custom evals · annotation)"]
    end
    T --> M --> D --> E --> EV
    EV -- Deploy --> T
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart LR
    subgraph Online["Online Evaluation"]
        T["Trace\n(traces · sessions · agents · prompts)"]
        M["Monitor\n(dashboards · LLM-as-judge · feedback)"]
    end
    subgraph Offline["Offline Evaluation"]
        D["Build Datasets\n(datasets · features-as-tests)"]
        E["Experiment\n(prompts · models · code variants)"]
        EV["Evaluate\n(judges · custom evals · annotation)"]
    end
    T --> M --> D --> E --> EV
    EV -- Deploy --> T
Loading

Reviews (1): Last reviewed commit: "docs: replace evaluation loop image with..." | Re-trigger Greptile

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 4, 2026
@vercel

vercel Bot commented Jul 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
langfuse-docs Ready Ready Preview, Comment Jul 4, 2026 2:19pm

Request Review

@dosubot dosubot Bot added the documentation Improvements or additions to documentation label Jul 4, 2026
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown

@claude review

@Lotte-Verheyden Lotte-Verheyden enabled auto-merge July 4, 2026 14:17
@dosubot dosubot Bot added the auto-merge This PR is set to be merged label Jul 4, 2026

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM — swaps a static image for the already-registered LoopDiagram MDX component used across the academy.

Extended reasoning...

Overview

Single-file docs change to content/docs/evaluation/core-concepts.mdx: removes a <Frame>-wrapped static PNG and replaces it with <LoopDiagram />, an existing component globally registered in mdx-components.tsx and already rendered on ~20 other MDX pages (academy and blog).

Security risks

None. Pure MDX content swap; no auth, data flow, or executable code paths touched.

Level of scrutiny

Minimal — this is a size:XS documentation-only edit labeled auto-merge. The component already exists and is battle-tested elsewhere in the same site.

Other factors

Bug hunting system found no issues. Vercel preview build is running. The only observation worth mentioning is that the PR description contains an empty backticks pair (``) where the component name was presumably intended — cosmetic only, doesn't affect the code.

@Lotte-Verheyden Lotte-Verheyden added this pull request to the merge queue Jul 4, 2026
Merged via the queue into main with commit 00d476d Jul 4, 2026
18 checks passed
@Lotte-Verheyden Lotte-Verheyden deleted the replace-eval-loop-image branch July 4, 2026 14:25
@dosubot dosubot Bot removed the auto-merge This PR is set to be merged label Jul 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant