Skip to content

feat(highcharts): implement pie-basic#157

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

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

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude Bot commented Dec 3, 2025

Summary

Implements pie-basic for highcharts library.

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

Implementation

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

Features

  • Basic pie chart with proportional slices
  • PyPlots style guide colors (Python Blue, Yellow, Red, etc.)
  • All optional parameters from spec:
    • colors: Custom color palette
    • startangle: Starting angle for first slice
    • explode: Offset distances for each slice
    • shadow: 3D shadow effect
    • legend: Toggle legend display
    • legend_loc: Legend positioning
    • autopct: Percentage label format
  • Proper input validation (empty data, missing columns, negative values, zero sum)
  • Google-style docstrings with type hints
  • PNG export via Selenium headless Chrome

Testing

  • Tested with sample data from spec
  • Verified plot.png output (1600x761 PNG)
  • Passed ruff formatting and linting

Add basic pie chart implementation for Highcharts library.

- Implements all optional parameters from spec (colors, startangle, explode, legend, etc.)
- Uses PyPlots style guide colors (#306998, #FFD43B, etc.)
- Includes proper input validation for empty data, missing columns, negative values
- Exports via Selenium headless Chrome with inline Highcharts JS
- Follows project patterns from existing highcharts implementations
@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 fc536d7 into plot/pie-basic Dec 3, 2025
10 checks passed
@github-actions github-actions Bot deleted the auto/pie-basic/highcharts 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