[scatter-basic] plotly implementation

[github-actions]

[scatter-basic] plotly Implementation

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

---

Attempt History

Attempts will be documented below as comments.

[github-actions]

Attempt 1/3

Technical Approach

• Imports: import numpy as np, import plotly.graph_objects as go
• Plot function: go.Scatter
• Config: template=plotly_white, color=white

Status

• PR: feat(plotly): implement scatter-basic #413
• File: plots/plotly/scatter/scatter-basic/default.py
• Workflow: 19997967892

---

[github-actions]

🧪 Test Results (PR #413)

Python:

📊 Plot Preview (Python 3.13)

plotly (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

Visual Quality

• VQ-001: Axes labeled correctly - "X Value" and "Y Value" are meaningful labels
• VQ-002: Visual clarity - Grid is subtle (rgba 0.1 opacity), doesn't dominate data
• VQ-003: Elements clear - Markers size 12 with 0.7 opacity, clearly visible
• VQ-004: No overlap - Labels, ticks, and title are well-spaced with proper margins
• VQ-005: Legend present - N/A (single series, no legend needed)
• VQ-006: Colorblind safe - Uses Python Blue (#306998), no red-green issues
• VQ-007: Image size - Correct 4800x2700 px (1600x900 with scale=3)
• VQ-008: Title - "Basic Scatter Plot" centered and clear

Code Quality

• CQ-001: Type hints - N/A for plot scripts (KISS style per guidelines)
• CQ-002: Docstring - N/A for plot scripts (KISS style per guidelines)
• CQ-003: Input validation - N/A for plot scripts (KISS style per guidelines)
• CQ-004: Clear error messages - N/A for plot scripts (KISS style per guidelines)
• CQ-005: No magic numbers - N/A for plot scripts (KISS style per guidelines)

Correctness

• CR-001: Data accurate - X and Y correctly mapped, correlation visible
• CR-002: Spec compliance - Basic scatter with (x, y) coordinates as specified
• CR-003: Edge cases - N/A for plot scripts with generated data

Issues Found

None - implementation follows all requirements.

Positive Highlights

1. Clean implementation following plotly library guidelines
2. Uses go.Figure() and go.Scatter as recommended in prompts/library/plotly.md
3. Proper plotly_white template for clean appearance
4. Font sizes (40pt title/axis, 32pt ticks) appropriate for high-res output
5. Subtle grid with 0.1 opacity doesn't distract from data
6. Proper margins ensure all labels are visible
7. Follows KISS style - no functions, no classes, just imports → data → plot → save

AI Feedback for Next Attempt

No changes needed - implementation is complete and meets all criteria.

Verdict: APPROVED