Skip to content

feat(seaborn): implement pie-basic#358

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

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

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 7, 2025

Summary

Implements pie-basic for seaborn library.

Parent Issue: #206
Sub-Issue: #243
Base Branch: plot/pie-basic
Attempt: 1/3

Implementation

  • plots/seaborn/pie/pie-basic/default.py

Notes

  • Seaborn does not have a native pie chart function
  • Uses matplotlib's ax.pie() with seaborn's styling context (sns.set_theme(style="white")) for consistent aesthetics
  • Follows KISS script style (no functions, no type hints)
  • Uses PyPlots.ai color palette
  • Configured for 16:9 aspect ratio output at 300 DPI
  • Font sizes follow style guide (title: 20pt, legend/labels: 16pt)

Simplify pie chart implementation following KISS principles:
- Remove function-based structure
- Use sequential script approach like gallery examples
- Apply 16:9 figsize for 4800x2700px output at 300 DPI
- Follow PyPlots.ai style guide font sizes
- Keep seaborn styling context for consistent aesthetics

Note: Seaborn lacks native pie chart support, so this uses
matplotlib's pie with seaborn styling.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 7, 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.

@claude claude bot added the ai-approved Quality OK, ready for merge label Dec 7, 2025
@github-actions github-actions bot merged commit 10f5a58 into plot/pie-basic Dec 7, 2025
8 checks passed
@github-actions github-actions bot deleted the auto/pie-basic/seaborn branch December 7, 2025 00:53
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants