feat(seaborn): implement surface-basic

[github-actions]

Implementation: surface-basic - seaborn

Implements the seaborn version of surface-basic.

File: plots/surface-basic/implementations/seaborn.py

Parent Issue: #1013

---

🤖 impl-generate workflow

[claude]

AI Review - Attempt 1/3

Image Description

The plot displays a 3D surface visualization of a mathematical function. The surface shows a central dome/peak rising from a flat base, resembling a damped sinc-like function. The viridis colormap is used, with purple/dark blue colors at the base (z ≈ 0) transitioning through green to yellow at the peak (z ≈ 0.85). The X and Y axes range from -4 to 4, while the Z axis ranges from approximately 0 to 0.8. The title "surface-basic · seaborn · pyplots.ai" is displayed at the top. A colorbar labeled "Z Value" is positioned on the right side showing the value scale. The viewing angle provides a good perspective of the 3D structure with elevation ~30° and azimuth ~45°. The grid is subtle and helps with depth perception. All axis labels (X Axis, Y Axis, Z Axis) are clearly visible.

Quality Score: 92/100

Criteria Checklist

Spec Compliance (35 pts) - 33/35

• SC-01: Correct plot type (10/10) - This is a proper 3D surface plot
• SC-02: Data mapped correctly (7/7) - X, Y grid with Z height values correctly mapped
• SC-03: Required features present (7/7) - Has colormap, colorbar, axis labels, smooth surface
• SC-04: Data range (4/4) - 40x40 grid within spec's 30-50 range, all data visible
• [-] SC-05: Legend accuracy (2/4) - Colorbar present but axis labels are generic ("X Axis", "Y Axis", "Z Axis") rather than context-specific
• SC-06: Title format (3/3) - Uses correct format: "surface-basic · seaborn · pyplots.ai"

Visual Quality (35 pts) - 33/35

• VQ-01: Meaningful axis labels (5/7) - Labels present but generic; could describe the mathematical function
• VQ-02: No overlapping text (6/6) - All text is readable with no overlaps
• VQ-03: Color choice (5/5) - Viridis colormap is excellent, colorblind-safe
• VQ-04: Clear data elements (5/5) - Surface is smooth, clear, with good alpha and antialiasing
• VQ-05: Layout balance (5/5) - Well-proportioned, no cut-off content
• VQ-06: Grid subtlety (3/3) - Grid is subtle and enhances depth perception
• VQ-07: Legend placement (2/2) - Colorbar is well-placed, doesn't obscure data
• VQ-08: Image size (2/2) - 4800x2700 px at 16:9 ratio (16300, 9300)

Data Quality (15 pts) - 14/15

• DQ-01: Feature coverage (6/6) - Shows height variation, smooth surface, interesting mathematical structure
• DQ-02: Realistic context (4/5) - Mathematical function (damped ripple) is appropriate for surface demo
• DQ-03: Appropriate scale (4/4) - 40x40 grid, sensible x/y range (-4 to 4), z values 0-0.85

Code Quality (15 pts) - 12/15

• CQ-01: KISS structure (4/4) - Simple sequential flow: imports → data → plot → save
• CQ-02: Reproducible (3/3) - Deterministic mathematical function, no random seed needed
• CQ-03: Library idioms (2/3) - Uses matplotlib 3D projection correctly; seaborn is only used for set_theme() styling
• CQ-04: Clean imports (2/2) - All imports are used
• CQ-05: Helpful comments (1/1) - Comments explain data and sizing choices
• CQ-06: No deprecated API (0/1) - Uses fig.add_subplot(111, projection='3d') which is fine, but matplotlib seaborn integration is minimal
• CQ-07: Output correct (1/1) - Saves as plot.png with correct dpi

Issues Found

1. Minor Issue: Generic axis labels ("X Axis", "Y Axis", "Z Axis") - While functional, more descriptive labels related to the mathematical function would enhance understanding
2. Minor Issue: Seaborn is only used for set_theme() styling - The 3D surface is rendered entirely by matplotlib's mplot3d. This is acceptable since seaborn doesn't have native 3D support, and using seaborn's theming is a valid approach

AI Feedback

The implementation successfully creates a 3D surface plot that meets the specification requirements. The mathematical function sin(sqrt(x²+y²)) * exp(-0.1*(x²+y²)) creates an interesting damped ripple pattern that effectively demonstrates surface visualization capabilities. The viridis colormap choice is excellent for colorblind accessibility. The code follows KISS principles and is well-structured.

Suggestion for future improvement: Consider using more descriptive axis labels that relate to the application (e.g., if showing terrain data, use "Latitude", "Longitude", "Elevation"). However, for a basic/generic surface plot demonstration, the current labels are acceptable.

Verdict: APPROVED