Skip to content

feat(letsplot): implement pie-basic#377

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

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

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 7, 2025

Summary

Implements pie-basic for letsplot library.

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

Implementation

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

Details

  • Uses lets-plot's geom_pie() with stat='identity' for pre-computed values
  • Applies PyPlots.ai color palette via scale_fill_manual()
  • Output dimensions: 4800×2700px (1600×900 base scaled 3x)
  • Includes title, legend with proper theming using theme_void() + custom theme()

Add basic pie chart implementation for lets-plot library using geom_pie.
- Uses PyPlots.ai color palette
- Follows style guide with 4800x2700px output (1600x900 scaled 3x)
- Includes title and legend with proper theming
@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 b9fb574 into plot/pie-basic Dec 7, 2025
8 checks passed
@github-actions github-actions bot deleted the auto/pie-basic/letsplot branch December 7, 2025 00:58
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