chore(plotly): update quality score 87 and review feedback for slope-basic

Author: github-actions[bot]

plots/slope-basic/implementations/python/plotly.py

@@ -1,7 +1,7 @@
-"""anyplot.ai
+""" anyplot.ai
 slope-basic: Basic Slope Chart (Slopegraph)
-Library: plotly | Python 3.13
-Quality: pending | Updated: 2026-04-30
+Library: plotly 6.7.0 | Python 3.13.13
+Quality: 87/100 | Updated: 2026-04-30
 """
 
 import os

plots/slope-basic/metadata/python/plotly.yaml

@@ -1,11 +1,8 @@
-# Per-library metadata for plotly implementation of slope-basic
-# Auto-generated by impl-generate.yml
-
 library: plotly
 language: python
 specification_id: slope-basic
 created: '2025-12-23T20:45:26Z'
-updated: '2026-04-30T16:39:15Z'
+updated: '2026-04-30T16:46:14Z'
 generated_by: claude-sonnet
 workflow_run: 25177301713
 issue: 981
@@ -15,7 +12,235 @@ preview_url_light: https://storage.googleapis.com/anyplot-images/plots/slope-bas
 preview_url_dark: https://storage.googleapis.com/anyplot-images/plots/slope-basic/python/plotly/plot-dark.png
 preview_html_light: https://storage.googleapis.com/anyplot-images/plots/slope-basic/python/plotly/plot-light.html
 preview_html_dark: https://storage.googleapis.com/anyplot-images/plots/slope-basic/python/plotly/plot-dark.html
-quality_score: null
+quality_score: 87
 review:
-  strengths: []
-  weaknesses: []
+  strengths:
+  - 'Perfect spec compliance: all required slopegraph features present (bidirectional
+    labels, directional color coding, time-point axis labels, appropriate entity count)'
+  - 'Perfect data and code quality: realistic business scenario, fully deterministic,
+    clean KISS code structure'
+  - Solid semantic color encoding using Okabe-Ito positions 1 and 2 with meaningful
+    directional assignment
+  - Both theme renders pass readability check; INK tokens correctly thread through
+    title, ticks, and axis labels
+  weaknesses:
+  - 'Label overlap in dense value regions: bottom-left cluster (Monitor Stand $55K
+    / Webcam HD $45K / USB Hub $30K) and right-side pair (Webcam HD $95K / USB Hub
+    $85K) overlap at 4800x2700 resolution'
+  - 'No visual emphasis on standout data points (strongest gainer: Laptop Pro +$65K;
+    biggest loser: Monitor Stand -$15K) — could add bold line or slight weight increase
+    to top/bottom performers'
+  - Axis lines (linecolor=INK_SOFT) add slight clutter; removing them would achieve
+    a cleaner slopegraph aesthetic
+  image_description: |-
+    Light render (plot-light.png):
+      Background: Warm off-white, consistent with #FAF8F1 — not pure white
+      Chrome: Title "Product Sales Q1 vs Q4 · slope-basic · plotly · anyplot.ai" in dark ink, clearly readable; y-axis label "Sales ($K)" in dark ink; tick labels in INK_SOFT gray; x-axis ticks show "Q1 2024" / "Q4 2024"
+      Data: 10 slope lines — 7 green (#009E73) for increases, 3 orange (#D55E00) for decreases; markers at endpoints; labels on both sides colored to match line direction; first categorical direction (increases) uses #009E73
+      Legibility verdict: PASS — all text readable, minor label crowding at bottom-left cluster and right-side pair but individually legible
+
+    Dark render (plot-dark.png):
+      Background: Near-black, consistent with #1A1A17 — not pure black
+      Chrome: Title appears in light #F0EFE8 ink, clearly readable against dark surface; axis labels and ticks in #B8B7B0 (INK_SOFT dark), visibly lighter than background; no dark-on-dark failures observed
+      Data: Green and orange line colors are identical to light render — Okabe-Ito data colors unchanged between themes as required; subtle light-on-dark grid lines visible
+      Legibility verdict: PASS — all text readable in dark theme; INK tokens correctly applied to all chrome elements
+  criteria_checklist:
+    visual_quality:
+      score: 27
+      max: 30
+      items:
+      - id: VQ-01
+        name: Text Legibility
+        score: 8
+        max: 8
+        passed: true
+        comment: Title 28px, axis labels 22px, tick labels 18px, annotations 16px
+          — all explicitly set, readable in both themes
+      - id: VQ-02
+        name: No Overlap
+        score: 3
+        max: 6
+        passed: false
+        comment: Bottom-left cluster (Monitor Stand/Webcam HD/USB Hub) and right-side
+          pair (Webcam HD/USB Hub at 10K gap) produce label collisions at 4800x2700
+      - id: VQ-03
+        name: Element Visibility
+        score: 6
+        max: 6
+        passed: true
+        comment: Lines width=3, markers size=14, perfectly adapted for 10-entity slope
+          chart
+      - id: VQ-04
+        name: Color Accessibility
+        score: 2
+        max: 2
+        passed: true
+        comment: Green vs. vermillion are CVD-safe and distinguish well by luminance
+      - id: VQ-05
+        name: Layout & Canvas
+        score: 4
+        max: 4
+        passed: true
+        comment: 220px side margins for labels; chart occupies ~72% canvas width,
+          appropriate for slopegraph
+      - id: VQ-06
+        name: Axis Labels & Title
+        score: 2
+        max: 2
+        passed: true
+        comment: Y-axis labeled Sales ($K) with units; x-axis uses Q1 2024 / Q4 2024
+          tick labels
+      - id: VQ-07
+        name: Palette Compliance
+        score: 2
+        max: 2
+        passed: true
+        comment: 'Increases #009E73, decreases #D55E00; backgrounds #FAF8F1 light
+          / #1A1A17 dark; all chrome theme-adaptive'
+    design_excellence:
+      score: 13
+      max: 20
+      items:
+      - id: DE-01
+        name: Aesthetic Sophistication
+        score: 5
+        max: 8
+        passed: false
+        comment: Semantic color coding and warm background above default, but lacks
+          publication-level polish — no emphasis on standouts
+      - id: DE-02
+        name: Visual Refinement
+        score: 4
+        max: 6
+        passed: false
+        comment: Subtle y-axis grid, no x-axis grid, generous margins; axis lines
+          retained add slight visual noise
+      - id: DE-03
+        name: Data Storytelling
+        score: 4
+        max: 6
+        passed: false
+        comment: Green/orange coding immediately communicates direction; crossed lines
+          reveal rank changes; overall positive trend visible
+    spec_compliance:
+      score: 15
+      max: 15
+      items:
+      - id: SC-01
+        name: Plot Type
+        score: 5
+        max: 5
+        passed: true
+        comment: Correct slopegraph with lines connecting two time points
+      - id: SC-02
+        name: Required Features
+        score: 4
+        max: 4
+        passed: true
+        comment: Labels at both endpoints, directional color coding, time-point axis
+          labels, 10 entities in 5-15 range
+      - id: SC-03
+        name: Data Mapping
+        score: 3
+        max: 3
+        passed: true
+        comment: X maps to Q1/Q4 time points, Y maps to sales values, all 10 products
+          visible
+      - id: SC-04
+        name: Title & Legend
+        score: 3
+        max: 3
+        passed: true
+        comment: Title contains slope-basic · plotly · anyplot.ai; no legend needed
+          with labeled endpoints
+    data_quality:
+      score: 15
+      max: 15
+      items:
+      - id: DQ-01
+        name: Feature Coverage
+        score: 6
+        max: 6
+        passed: true
+        comment: 7 increasing + 3 decreasing products; shows rank changes, crossing
+          lines, varied magnitudes
+      - id: DQ-02
+        name: Realistic Context
+        score: 5
+        max: 5
+        passed: true
+        comment: Product sales Q1 vs Q4 — neutral tech business scenario, no controversial
+          content
+      - id: DQ-03
+        name: Appropriate Scale
+        score: 4
+        max: 4
+        passed: true
+        comment: Sales $30K-$310K per quarter for tech products is realistic and proportionally
+          sound
+    code_quality:
+      score: 10
+      max: 10
+      items:
+      - id: CQ-01
+        name: KISS Structure
+        score: 3
+        max: 3
+        passed: true
+        comment: Imports → Tokens → Data → Plot → Style → Save; no functions or classes
+      - id: CQ-02
+        name: Reproducibility
+        score: 2
+        max: 2
+        passed: true
+        comment: All data hardcoded and deterministic
+      - id: CQ-03
+        name: Clean Imports
+        score: 2
+        max: 2
+        passed: true
+        comment: Only os and plotly.graph_objects imported, both used
+      - id: CQ-04
+        name: Code Elegance
+        score: 2
+        max: 2
+        passed: true
+        comment: Clean and Pythonic; two annotation loops slightly repetitive but
+          acceptable
+      - id: CQ-05
+        name: Output & API
+        score: 1
+        max: 1
+        passed: true
+        comment: Saves plot-{THEME}.png and plot-{THEME}.html correctly
+    library_mastery:
+      score: 7
+      max: 10
+      items:
+      - id: LM-01
+        name: Idiomatic Usage
+        score: 4
+        max: 5
+        passed: true
+        comment: go.Scatter with lines+markers, add_annotation, hovertemplate with
+          <extra></extra> suppression shows Plotly expertise
+      - id: LM-02
+        name: Distinctive Features
+        score: 3
+        max: 5
+        passed: false
+        comment: Interactive hovertemplate with dollar-formatted info and HTML export
+          via write_html leverages Plotly's web-native capabilities
+  verdict: REJECTED
+impl_tags:
+  dependencies: []
+  techniques:
+  - annotations
+  - hover-tooltips
+  - html-export
+  - manual-ticks
+  patterns:
+  - iteration-over-groups
+  dataprep: []
+  styling:
+  - grid-styling