[scatter-basic] seaborn implementation

[github-actions]

[scatter-basic] seaborn Implementation

Parent Issue: #207
Spec: specs/scatter-basic.md
Library: seaborn
Feature Branch: plot/scatter-basic

---

Attempt History

Attempts will be documented below as comments.

[github-actions]

Attempt 1/3

Technical Approach

• Imports: import matplotlib.pyplot as plt, import numpy as np, import pandas as pd
• Plot function: sns.scatterplot
• Config: figsize=16, 9, color=#306998

Status

• PR: feat(seaborn): implement scatter-basic #353
• File: plots/seaborn/scatterplot/scatter-basic/default.py
• Workflow: 19995640490

---

[github-actions]

🧪 Test Results (PR #353)

Python:

📊 Plot Preview (Python 3.13)

seaborn (default) - UPDATE

Before	After

View version history (2 versions)

---

🤖 Generated by gen-preview workflow

[claude]

AI Review - Attempt 1/3

Quality Evaluation

Evaluator	Score	Verdict
Claude	92/100	approve

Criteria Checklist

• VQ-001: Axes labeled correctly ("X Value", "Y Value")
• VQ-002: Grid is subtle (alpha=0.3)
• VQ-003: Elements clear (appropriate size=30, alpha=0.6)
• VQ-004: No overlap issues
• VQ-005: Legend N/A (single series, no hue mapping)
• VQ-006: Colorblind safe (steelblue)
• VQ-007: Image size correct (16:9 aspect ratio)
• VQ-008: Title centered and clear
• CQ-001: Type hints present (full typing)
• CQ-002: Docstring complete (Args, Returns, Raises, Example)
• CQ-003: Input validation (empty data, columns, numeric types)
• CQ-004: Clear error messages (lists available columns)
• CQ-005: No magic numbers (all exposed as parameters)
• CR-001: Data accurate
• CR-002: Spec compliance
• CR-003: Edge cases handled

Issues Found

None - implementation meets all quality criteria.

Strengths

1. Excellent input validation: Checks for empty data, missing columns, and non-numeric data types
2. Complete docstring: Google-style with all required sections including Example
3. Clean visual output: Subtle grid, appropriate point size and transparency for scatter data
4. Library-specific best practices: Uses axes-level API (sns.scatterplot with ax=ax) per seaborn guidelines
5. Flexible parameters: All styling options exposed with sensible defaults

AI Feedback for Next Attempt

No issues found - implementation is production ready.

Verdict: APPROVED