Skip to content

feat(seaborn): implement box-basic#299

Merged
github-actions[bot] merged 1 commit intoplot/box-basicfrom
auto/box-basic/seaborn
Dec 6, 2025
Merged

feat(seaborn): implement box-basic#299
github-actions[bot] merged 1 commit intoplot/box-basicfrom
auto/box-basic/seaborn

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 6, 2025

Summary

Implements box-basic for seaborn library.

Parent Issue: #203
Sub-Issue: #220
Base Branch: plot/box-basic
Attempt: 1/3

Implementation

  • plots/seaborn/boxplot/box-basic/default.py

Changes

  • Simplified implementation to follow KISS principles (sequential script, no functions)
  • Uses style guide color palette (#306998, #FFD43B, #DC2626, #059669)
  • Follows seaborn 0.14+ API (uses hue with palette, legend=False)
  • Uses spec-defined sample data (4 groups with different distributions)
  • Properly sized output (16:9 aspect ratio, 300 DPI)

Simplify implementation to follow KISS principles:
- Remove function wrapper, use sequential script style
- Use style guide color palette
- Follow seaborn 0.14+ API (hue with palette, legend=False)
- Use spec-defined sample data
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 6, 2025

✅ Plot Tests Passed

Python 3.13 (required): passed

Compatibility: 3.12 3.13

Note: Only Python 3.13 is required to pass. Python 3.12 is tested for compatibility.

@github-actions github-actions bot added the ai-review-failed AI review action failed or timed out label Dec 6, 2025
@claude claude bot added the ai-approved Quality OK, ready for merge label Dec 6, 2025
@github-actions github-actions bot merged commit ada0895 into plot/box-basic Dec 6, 2025
8 checks passed
@github-actions github-actions bot deleted the auto/box-basic/seaborn branch December 6, 2025 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-approved Quality OK, ready for merge ai-review-failed AI review action failed or timed out

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants