Skip to content

feat(plotly): implement pie-basic#155

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

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

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude Bot commented Dec 3, 2025

Summary

Implements pie-basic for plotly library.

  • Implements basic pie chart using Plotly's go.Pie trace
  • Uses PyPlots.ai color palette for consistent visual identity
  • Supports all optional parameters from spec (colors, explode, legend, startangle, etc.)

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

Implementation

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

Features

  • Input validation for empty data, missing columns, and negative values
  • Customizable percentage label formatting via autopct parameter
  • Legend positioning with matplotlib-to-Plotly location mapping
  • Circular aspect ratio enforcement for proper visualization
  • Interactive hover tooltips showing label, value, and percentage
  • Configurable slice explosion for emphasis
  • White slice borders for clear visual separation

Add basic pie chart implementation for Plotly library with:
- PyPlots.ai color palette for consistent styling
- Full parameter support from spec (colors, explode, legend, etc.)
- Input validation for empty data, missing columns, negative values
- Percentage label formatting with configurable autopct
- Legend positioning with matplotlib-to-Plotly location mapping
- Circular aspect ratio enforcement
- Interactive hover tooltips
@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 e279ba3 into plot/pie-basic Dec 3, 2025
10 checks passed
@github-actions github-actions Bot deleted the auto/pie-basic/plotly branch December 3, 2025 22:25
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