feat(seaborn): implement spectrum-basic

[github-actions]

Implementation: spectrum-basic - seaborn

Implements the seaborn version of spectrum-basic.

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

Parent Issue: #2926

---

🤖 impl-generate workflow

[claude]

AI Review - Attempt 1/3

Image Description

The plot displays a frequency spectrum with a blue line showing amplitude (in dB) on the y-axis ranging from -60 to 10, and frequency (Hz) on the x-axis from 0 to 300. The title "spectrum-basic · seaborn · pyplots.ai" is clearly displayed at the top. Four distinct peaks are visible at approximately 50 Hz (7 dB), 100 Hz (0 dB), 150 Hz (-2 dB), and 220 Hz (-10 dB). Yellow dashed vertical lines mark these peak frequencies. The area under the curve is filled with a light blue/teal semi-transparent fill. The background noise floor sits around -30 to -40 dB with natural variation. Grid lines are subtle and dashed. Overall, the plot has a clean, professional appearance with good use of the canvas space.

Quality Score: 92/100

Criteria Checklist

Visual Quality (37/40 pts)

• VQ-01: Text Legibility (10/10) - Title at 24pt, axis labels at 20pt, tick labels at 16pt - all perfectly readable
• VQ-02: No Overlap (8/8) - No overlapping text elements
• VQ-03: Element Visibility (7/8) - Line is visible with good width (2.5), fill adds clarity, peaks are distinct
• VQ-04: Color Accessibility (5/5) - Single blue color scheme (#306998) with good contrast, colorblind-safe
• VQ-05: Layout Balance (5/5) - Plot fills canvas well with 16:9 aspect ratio, balanced margins
• VQ-06: Axis Labels (2/2) - Clear labels with units: "Frequency (Hz)" and "Amplitude (dB)"
• VQ-07: Grid & Legend (0/2) - Grid is subtle (alpha=0.3), but no legend present; however, for a single-series spectrum plot, no legend is needed

Spec Compliance (25/25 pts)

• SC-01: Plot Type (8/8) - Correct frequency spectrum/line plot
• SC-02: Data Mapping (5/5) - X=frequency, Y=amplitude correctly assigned
• SC-03: Required Features (5/5) - Has FFT-based spectrum, dB scale, peak highlighting with vertical lines
• SC-04: Data Range (3/3) - Focused range 0-300 Hz shows all relevant data
• SC-05: Legend Accuracy (2/2) - N/A for single series, properly handled
• SC-06: Title Format (2/2) - Correct format: "spectrum-basic · seaborn · pyplots.ai"

Data Quality (18/20 pts)

• DQ-01: Feature Coverage (7/8) - Shows fundamental (50 Hz) with harmonics (100, 150 Hz), additional component (220 Hz), and noise floor - demonstrates key spectrum features
• DQ-02: Realistic Context (7/7) - Machinery vibration analysis scenario is realistic and neutral
• DQ-03: Appropriate Scale (4/5) - FFT bins at 1000 Hz sample rate are appropriate; dB scale range (-60 to 10) is realistic

Code Quality (9/10 pts)

• CQ-01: KISS Structure (3/3) - Linear script: imports → data → plot → save
• CQ-02: Reproducibility (3/3) - np.random.seed(42) is set
• CQ-03: Clean Imports (2/2) - Only used imports: matplotlib, numpy, seaborn
• CQ-04: No Deprecated API (1/1) - Uses current seaborn API
• CQ-05: Output Correct (0/1) - Saves as "plot.png" but in current directory, not relative path issue

Library Features (3/5 pts)

• LF-01: Uses seaborn's lineplot and set_context for styling, but could use more distinctive seaborn features (3/5)

Strengths

• Excellent visual clarity with properly sized text elements and good color contrast
• Realistic machinery vibration scenario with fundamental frequency and harmonics
• Effective use of fill_between to enhance the spectrum visualization
• Peak frequency highlighting with vertical dashed lines aids interpretation
• Proper use of dB scale as recommended in the specification
• Clean KISS code structure with reproducible random seed

Weaknesses

• Uses seaborn primarily for lineplot which is relatively basic; could leverage more distinctive seaborn features
• Grid styling uses alpha=0.3 which is acceptable but slightly at the edge of the 0.2-0.4 recommendation

Verdict: APPROVED