feat(seaborn): implement waffle-basic

[github-actions]

Implementation: waffle-basic - python/seaborn

Implements the python/seaborn version of waffle-basic.

File: plots/waffle-basic/implementations/python/seaborn.py

Parent Issue: #998

---

🤖 impl-generate workflow

[claude]

AI Review - Attempt 1/3

Image Description

Light render (plot-light.png): Warm off-white (#FAF8F1) background. Bold dark title "waffle-basic · seaborn · anyplot.ai" at top, with a muted-gray caption "Each square = 1% of 500 survey respondents" beneath it. A 10×10 grid of colored squares (no axes/spines) represents the waffle chart, with thin off-white gutters between squares. A two-row legend below lists each category with its percentage. All text is dark ink on the light surface and clearly readable.

Dark render (plot-dark.png): Warm near-black (#1A1A17) background. The same title and caption now render in light ink, fully legible. The legend text is also light-colored and readable. The five data colors (brand green, lavender, blue, ochre, matte red) are pixel-identical to the light render — only the chrome flipped as expected.

Both renders pass the theme-readability check: no dark-on-dark or light-on-light failures in either image.

Score: 89/100

Category	Score	Max
Visual Quality	29	30
Design Excellence	15	20
Spec Compliance	13	15
Data Quality	15	15
Code Quality	10	10
Library Mastery	7	10
Total	89	100

Visual Quality (29/30)

• VQ-01: Text Legibility (7/8)
• VQ-02: No Overlap (6/6)
• VQ-03: Element Visibility (6/6)
• VQ-04: Color Accessibility (2/2)
• VQ-05: Layout & Canvas (4/4)
• VQ-06: Axis Labels & Title (2/2)
• VQ-07: Palette Compliance (2/2)

Design Excellence (15/20)

• DE-01: Aesthetic Sophistication (6/8) - Custom palette and deliberate typography hierarchy, clearly above defaults but not fully publication-grade
• DE-02: Visual Refinement (5/6) - All spines removed, floating-grid gutters, generous whitespace, frameless legend
• DE-03: Data Storytelling (4/6) - Bottom-up "filling a glass" fill order anchors the dominant category and creates hierarchy, but no explicit callout

Spec Compliance (13/15)

• SC-01: Plot Type (5/5)
• SC-02: Required Features (4/4)
• SC-03: Data Mapping (3/3)
• SC-04: Title & Legend (1/3) - Title is missing the mandated language segment

Data Quality (15/15)

• DQ-01: Feature Coverage (6/6)
• DQ-02: Realistic Context (5/5)
• DQ-03: Appropriate Scale (4/4)

Code Quality (10/10)

• CQ-01: KISS Structure (3/3)
• CQ-02: Reproducibility (2/2)
• CQ-03: Clean Imports (2/2)
• CQ-04: Code Elegance (2/2)
• CQ-05: Output & API (1/1)

Library Mastery (7/10)

• LM-01: Idiomatic Usage (4/5)
• LM-02: Distinctive Features (3/5) - Repurposing sns.heatmap for a waffle grid is clever but the underlying primitive is generic

Score Caps Applied

• None

Strengths

• Clever repurposing of sns.heatmap with a ListedColormap and manual grid construction to render a native seaborn waffle chart, since the library has no dedicated waffle function
• Floating-grid effect (linecolor=PAGE_BG, linewidths=2.5) with all spines removed gives clean square separation without a heavy frame
• Deliberate bottom-to-top "filling a glass" fill order anchors the dominant category (Very Satisfied, 42%) at the base, giving the grid a visual hierarchy beyond a flat category dump
• Correct Imprint categorical palette in canonical order with brand green #009E73 first, identical across both theme renders
• Theme-adaptive chrome is fully correct in both renders — no dark-on-dark or light-on-light failures
• Legend cleanly lists each category with its percentage, satisfying the spec's requirement

Weaknesses

• Title omits the mandated language segment: it renders waffle-basic · seaborn · anyplot.ai but the required format is {spec-id} · {language} · {library} · anyplot.ai — should read waffle-basic · python · seaborn · anyplot.ai
• Design Excellence is solid but fairly minimal (flat color blocks, no secondary emphasis on the largest segment) — stops short of publication-grade polish
• Library Mastery is only moderate: sns.heatmap is a workaround for a chart type seaborn doesn't natively support — a smart trick, but still a generic underlying primitive

Issues Found

1. SC-04 HIGH: Title string is missing the required python language segment
   • Fix: change the fig.text(...) title literal from "waffle-basic · seaborn · anyplot.ai" to "waffle-basic · python · seaborn · anyplot.ai"
2. DE-01/DE-03 LOW: Composition is clean but minimal — no secondary emphasis calling out the dominant or extreme categories
   • Fix: consider a subtle emphasis technique (e.g. a slightly bolder outline or callout on the largest/smallest segment) to sharpen the storytelling, while keeping the design restrained

AI Feedback for Next Attempt

Fix the title first — it's currently missing the mandated python language segment between the spec-id and the library name. Everything else about this implementation is strong: keep the Imprint palette usage, the floating-grid styling, and the bottom-up fill order. If you want to push Design Excellence and Data Storytelling further, consider a subtle secondary emphasis on the dominant or extreme category (e.g., a slightly heavier outline) without adding clutter.

Verdict: REJECTED

[github-actions]

🔧 Repair Attempt 1/4

Applied fixes based on AI review feedback.

Status: Repair completed, re-triggering review...

---

🤖 impl-repair

[github-actions]

🔧 AI Review Produced No Score — Auto-Retrying

The Claude Code Action ran but didn't write quality_score.txt. Auto-retrying review once...

---

🤖 impl-review

[github-actions]

❌ AI Review Failed (auto-retry exhausted)

The AI review action completed but did not produce valid output files. Auto-retry already tried once.

What happened:

• The Claude Code Action ran
• No quality_score.txt file was created

Manual rerun:

gh workflow run impl-review.yml -f pr_number=9948

---

🤖 impl-review

[github-actions]

❌ AI Review Failed (auto-retry exhausted)

The AI review action completed but did not produce valid output files. Auto-retry already tried once.

What happened:

• The Claude Code Action ran
• No quality_score.txt file was created

Manual rerun:

gh workflow run impl-review.yml -f pr_number=9948

---

🤖 impl-review

[MarkusNeusinger]

Closing as superseded.

main already carries plots/waffle-basic/implementations/python/seaborn with quality_score: 85, which is at or above what this PR would land. impl-merge.yml has no regression guard against the score already on main, so resuming this PR would overwrite the better implementation with a worse one.

This PR stalled on the score-0 sentinel bug in impl-review.yml (a legitimate AR-08/AR-09 auto-reject was misreported as a crashed review, so impl-repair was never dispatched). That root cause is fixed in #10179, but the fix does not make this particular PR worth resuming — the coverage it would provide already exists at a higher score.

🤖 Closed during PR-queue cleanup.

[MarkusNeusinger]

Correction to the comment above: it was templated and its "at or above" claim does not hold for this PR specifically.

main carries waffle-basic/seaborn at 85; this PR reached 89, so it is the one stuck PR that would have improved main by 4 points.

It is closed deliberately anyway, for a different reason: its failure mode is the 2026-07-26 platform brownout (three consecutive re-reviews returned no output at all, no claude[bot] comment), not the score-0 sentinel bug fixed in #10179 — so a re-review has no identified mechanism behind it. And per the 2026-07 audit the score median already sits at 89 against a 72–78 target, which puts a 4-point delta on one of 15 libraries inside the noise band.

🤖 PR-queue cleanup.