feat(ggplot2): implement wireframe-3d-basic

[github-actions]

Implementation: wireframe-3d-basic - r/ggplot2

Implements the r/ggplot2 version of wireframe-3d-basic.

File: plots/wireframe-3d-basic/implementations/r/ggplot2.R

Parent Issue: #1015

---

🤖 impl-generate workflow

[claude]

AI Review - Attempt 1/3

Image Description

Light render (plot-light.png): Warm off-white (#FAF8F1) background. Title "wireframe-3d-basic · r · ggplot2 · anyplot.ai" centered at top in dark ink, clearly readable. A single ripple surface z = sin(sqrt(x²+y²)) rendered as a green (#009E73) wireframe mesh of diamond-shaped quads with painter's-algorithm hidden-line removal (nearer quads occlude farther mesh lines) and depth-based edge-opacity fading. A faint floor plane grounds the surface; X/Y/Z axis lines and bold axis labels sit in dark ink/gray. All chrome text is readable against the light background — except the "-6"/"-3" X/Y tick labels (mirrored on both sides of the Z axis), which sit directly inside the mesh's projected footprint with thin green mesh lines crossing through the glyphs. Text remains decipherable (good contrast) but the overlap is real and confirmed by close crop.

Dark render (plot-dark.png): Warm near-black (#1A1A17) background. Same title, same green (#009E73) wireframe mesh — data colors are pixel-identical to the light render, confirming only chrome flipped. Chrome (axis lines, "X"/"Y"/"Z" labels, tick labels) correctly switches to light gray/near-white with strong contrast — no dark-on-dark failures. The same tick-label/mesh overlap caveat from the light render applies here too (confirmed via cropped close-up), though legibility itself still holds.

Both renders viewed and analyzed above.

Score: 87/100

Category	Score	Max
Visual Quality	24	30
Design Excellence	17	20
Spec Compliance	15	15
Data Quality	14	15
Code Quality	9	10
Library Mastery	8	10
Total	87	100

Visual Quality (24/30)

• VQ-01: Text Legibility (7/8) — readable in both themes; mesh lines cross through "-6"/"-3" glyphs
• VQ-02: No Overlap (3/6) — confirmed: X/Y tick labels overlap mesh strands in both renders
• VQ-03: Element Visibility (5/6) — mesh density/line weight appropriate
• VQ-04: Color Accessibility (2/2) — single hue, CVD-safe
• VQ-05: Layout & Canvas (3/4) — no clipping, good margins; tick labels float mid-plot rather than clearly outside it
• VQ-06: Axis Labels & Title (2/2) — X/Y/Z labeled, title format exact
• VQ-07: Palette Compliance (2/2) — brand green sole data color, correct backgrounds, correct chrome flip

Design Excellence (17/20)

• DE-01: Aesthetic Sophistication (7/8) - Custom perspective projection + painter's-algorithm hidden-line removal + depth-fade
• DE-02: Visual Refinement (5/6) - theme_void chrome, subtle floor plane, generous whitespace
• DE-03: Data Storytelling (5/6) - z-exaggeration and hidden-line removal create a clear, solid-looking focal form

Spec Compliance (15/15)

• SC-01: Plot Type (5/5)
• SC-02: Required Features (4/4) — grid lines both directions, consistent line color, elev 30/azim 45, labeled axes with ticks
• SC-03: Data Mapping (3/3)
• SC-04: Title & Legend (3/3)

Data Quality (14/15)

• DQ-01: Feature Coverage (5/6) — grid_n=20 is at the low end of the spec's 20-50 recommended range
• DQ-02: Realistic Context (5/5) — matches the spec's own suggested ripple example
• DQ-03: Appropriate Scale (4/4)

Code Quality (9/10)

• CQ-01: KISS Structure (2/3) — small projection helper functions, justified by the 3D-projection workaround
• 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 (8/10)

• LM-01: Idiomatic Usage (4/5) — composes geom_polygon/geom_segment/geom_text + theme_void idiomatically for a scene ggplot2 has no native primitive for
• LM-02: Distinctive Features (4/5) - Full painter's-algorithm hidden-line removal implemented purely in ggplot2 grammar

Score Caps Applied

• None

Strengths

• Genuinely sophisticated 3D-in-2D engineering: manual perspective projection (elevation 30°, azimuth 45°) combined with painter's-algorithm hidden-line removal — a real technical achievement given ggplot2 has no native 3D grammar.
• Depth-based alpha fading on mesh edges adds a convincing depth cue without a second hue.
• Single-series brand green is pixel-identical between light/dark renders; only chrome flips correctly.
• All three axes clearly labeled with tick marks; title matches the required format exactly.
• Data (ripple function) matches the spec's suggested example precisely, with a sensible axis range.

Weaknesses

• X-axis and Y-axis tick labels ("-6" and "-3", mirrored on both sides) sit inside the wireframe's screen footprint with mesh strands crossing through the glyphs — confirmed by close crop in both renders. Push those tick anchors further outside the mesh's projected footprint (matching the offset technique already used for the "0"/"3"/"6" ticks and the Z-tick labels) so no tick text sits on top of mesh lines.
• Grid resolution (20x20) sits at the very bottom of the spec's recommended 20x20-50x50 range; a modest bump (e.g. 28-32) would add surface fidelity without hurting legibility now that hidden-line removal keeps the mesh readable.

Issues Found

1. VQ-02 MEDIUM: X/Y tick labels overlap wireframe mesh lines in both renders
   • Fix: Increase the offset used to place "-3"/"-6" tick label anchors (analogous to the existing z_ticks px offset trick) so they clear the mesh's projected footprint entirely.

AI Feedback for Next Attempt

This implementation is strong and does not require another attempt. If revisited: nudge the "-3"/"-6" X/Y tick label positions further outward so they clear the mesh's silhouette (reuse the same lateral-offset technique already applied to the Z-tick labels), and consider bumping grid_n from 20 toward 28-32 for slightly richer surface fidelity.

Verdict: APPROVED

[github-actions]

🔧 Repair Attempt 1/4

Applied fixes based on AI review feedback.

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

---

🤖 impl-repair

[claude]

AI Review - Attempt 2/3

Image Description

Light render (plot-light.png): Warm off-white (#FAF8F1) background. Title "wireframe-3d-basic · r · ggplot2 · anyplot.ai" centered at top in dark ink, clearly readable. A single ripple surface z = sin(sqrt(x²+y²)) rendered as a green (#009E73) wireframe mesh of diamond-shaped quads with painter's-algorithm hidden-line removal (nearer quads occlude farther mesh lines via opaque page-background fill) and depth-based edge-opacity fading. A faint floor plane grounds the surface; bold X/Y/Z axis labels and axis-box edges sit in dark ink/gray. Z-axis tick labels ("1"/"0"/"-1") sit cleanly clear of the mesh (verified via crop). The X/Y-axis "-6"/"-3" tick labels (mirrored on both sides of the Z axis), however, still have thin green mesh strands crossing directly through the glyphs — confirmed by a targeted pixel crop. Text remains decipherable, but the overlap flagged in Attempt 1 is still present.

Dark render (plot-dark.png): Warm near-black (#1A1A17) background. Same title, same green (#009E73) wireframe mesh — data colors are pixel-identical to the light render, confirming only chrome flipped. Chrome (axis-box lines, "X"/"Y"/"Z" labels, tick labels) correctly switches to light gray/near-white with strong contrast — no dark-on-dark failures anywhere. The same "-6"/"-3" tick-label/mesh overlap from the light render is present here too (confirmed via cropped close-up); legibility itself still holds.

Both renders viewed and analyzed above; the tick-label region was additionally cropped and pixel-inspected on both PNGs to confirm overlap status precisely.

Score: 87/100

Category	Score	Max
Visual Quality	23	30
Design Excellence	17	20
Spec Compliance	15	15
Data Quality	15	15
Code Quality	9	10
Library Mastery	8	10
Total	87	100

Visual Quality (23/30)

• VQ-01: Text Legibility (7/8) — readable in both themes; mesh lines cross through "-6"/"-3" glyphs
• VQ-02: No Overlap (2/6) — unresolved from Attempt 1: X/Y tick labels still overlap mesh strands in both renders despite the repair attempt
• VQ-03: Element Visibility (5/6) — mesh density/line weight appropriate for the denser 30x30 grid
• VQ-04: Color Accessibility (2/2) — single hue, CVD-safe
• VQ-05: Layout & Canvas (3/4) — no clipping, good margins; tick labels still float mid-plot rather than clearly outside the mesh
• VQ-06: Axis Labels & Title (2/2) — X/Y/Z labeled, title format exact
• VQ-07: Palette Compliance (2/2) — brand green sole data color, correct backgrounds, correct chrome flip

Design Excellence (17/20)

• DE-01: Aesthetic Sophistication (7/8) - Custom perspective projection + painter's-algorithm hidden-line removal + depth-fade
• DE-02: Visual Refinement (5/6) - theme_void chrome, subtle floor plane, generous whitespace
• DE-03: Data Storytelling (5/6) - z-exaggeration and hidden-line removal create a clear, solid-looking focal form

Spec Compliance (15/15)

• SC-01: Plot Type (5/5)
• SC-02: Required Features (4/4) — grid lines both directions, consistent line color, elev 30/azim 45, labeled axes with ticks
• SC-03: Data Mapping (3/3)
• SC-04: Title & Legend (3/3)

Data Quality (15/15)

• DQ-01: Feature Coverage (6/6) — grid_n raised from 20 to 30 this attempt, now solidly mid the spec's 20-50 recommended range
• DQ-02: Realistic Context (5/5) — matches the spec's own suggested ripple example
• DQ-03: Appropriate Scale (4/4)

Code Quality (9/10)

• CQ-01: KISS Structure (2/3) — small projection helper functions, justified by the 3D-projection workaround
• 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 (8/10)

• LM-01: Idiomatic Usage (4/5) — composes geom_polygon/geom_segment/geom_text + theme_void idiomatically for a scene ggplot2 has no native primitive for
• LM-02: Distinctive Features (4/5) - Full painter's-algorithm hidden-line removal implemented purely in ggplot2 grammar

Score Caps Applied

• None

Strengths

• Genuinely sophisticated 3D-in-2D engineering: manual perspective projection (elevation 30°, azimuth 45°) combined with painter's-algorithm hidden-line removal implemented purely in ggplot2 grammar.
• Grid resolution raised from 20x20 to 30x30 this attempt, now solidly mid-range within the spec's recommended window, with no legibility cost thanks to the opaque-fill occlusion trick.
• Depth-based alpha fading on mesh edges adds a convincing depth cue without a second hue.
• Single-series brand green is pixel-identical between light/dark renders; only chrome flips correctly, with no dark-on-dark or light-on-light failures.
• All three axes clearly labeled with tick marks; title matches the required format exactly.

Weaknesses

• The X/Y tick-label overlap flagged in Attempt 1 is still present after repair, confirmed by pixel-cropped close-up on both renders: the "-6" and "-3" labels (mirrored on both sides of the Z axis) still have wireframe mesh strands crossing directly through the glyphs. The applied fix (tick_clearance <- 5.5 px lateral offset) was too small to clear the mesh's silhouette, and bumping grid_n from 20 to 30 in the same commit made the local mesh denser, so roughly as many strands cross the labels as before. Increase tick_clearance substantially (try 15-20 instead of 5.5) and re-verify specifically at the "-6"/"-3" positions against the denser 30x30 mesh.
• The "-6"/"-3" tick labels still read as floating mid-plot rather than clearly outside the mesh silhouette (minor, secondary to the overlap itself).

Issues Found

1. VQ-02 MEDIUM: X/Y tick labels still overlap wireframe mesh lines in both renders — unresolved from Attempt 1
   • Fix: Increase tick_clearance well beyond 5.5 (try 15-20) so the "-3"/"-6" tick label anchors clear the mesh's projected footprint entirely, and re-check against the current 30x30 grid density rather than the original 20x20.

AI Feedback for Next Attempt

Score is high enough to approve, but the specific fix requested in Attempt 1 for the "-6"/"-3" tick-label overlap did not land — the offset applied was too small, especially now that the mesh is denser (grid_n 30). If revisited, increase tick_clearance to roughly 15-20 (from 5.5) and confirm with a pixel crop that no mesh strand crosses the "-6"/"-3" glyphs at the new grid density.

Verdict: APPROVED