Skip to content

feat(plotnine): implement pie-basic#370

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

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

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 7, 2025

Summary

Implements pie-basic for plotnine library.

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

Implementation

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

Details

  • Simplified to KISS pattern (no functions, sequential script)
  • Uses matplotlib directly since plotnine lacks coord_polar() support
  • Applies PyPlots.ai color palette
  • Uses 16:9 aspect ratio (figsize 16x9 at 300 dpi)
  • Legend positioned outside pie for clarity
  • Percentage labels displayed on each slice

Simplify pie chart implementation to follow KISS pattern:
- Remove function wrapper, use sequential script format
- Use matplotlib directly (plotnine lacks coord_polar)
- Apply PyPlots.ai color palette
- Use 16:9 aspect ratio (4800x2700 at 300 dpi)
@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-rejected Quality not OK, triggers update label Dec 7, 2025
@github-actions github-actions bot added ai-attempt-1 First repair attempt and removed ai-rejected Quality not OK, triggers update labels Dec 7, 2025
@github-actions github-actions bot merged commit b63611f into plot/pie-basic Dec 7, 2025
7 of 8 checks passed
@github-actions github-actions bot deleted the auto/pie-basic/plotnine branch December 7, 2025 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-attempt-1 First repair attempt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants