Skip to content

feat(altair): implement pie-basic#154

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

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

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude Bot commented Dec 3, 2025

Summary

Implements pie-basic for altair library.

Parent Issue: #142
Sub-Issue: #147
Base Branch: plot/pie-basic
Attempt: 1/3

Implementation

  • plots/altair/arc/pie-basic/default.py

Features

  • Basic pie chart using Altair's mark_arc()
  • Percentage labels on slices with configurable format
  • PyPlots.ai default color palette (#306998, #FFD43B, #DC2626, #059669, #8B5CF6, #F97316)
  • Legend with configurable position (right, left, top, bottom)
  • Support for donut style via inner_radius parameter
  • Configurable start angle (default: 90° - starts from 12 o'clock)
  • Tooltips showing category and value
  • Input validation for:
    • Empty data
    • Missing columns
    • Negative values
    • Zero sum values

Code Quality

  • Type hints on all parameters and return type
  • Google-style docstring with Args, Returns, Raises, Example
  • Follows PyPlots.ai style guide (colors, fonts, dimensions)

Add basic pie chart implementation for altair library using mark_arc().

Features:
- Percentage labels on slices with configurable format
- Customizable color palette (defaults to PyPlots.ai colors)
- Legend with configurable position
- Support for donut style via inner_radius parameter
- Configurable start angle
- Input validation for empty data, missing columns, and negative values
- Tooltips showing category and value
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 3, 2025

✅ Plot Tests Passed

Created with Python 3.14

Compatibility: 3.11 3.12 3.13

@claude claude Bot added the ai-approved Quality OK, ready for merge label Dec 3, 2025
@github-actions github-actions Bot merged commit 7309a12 into plot/pie-basic Dec 3, 2025
10 checks passed
@github-actions github-actions Bot deleted the auto/pie-basic/altair branch December 3, 2025 22:26
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