Skip to content

feat(bokeh): implement pie-basic#365

Merged
github-actions[bot] merged 2 commits intoplot/pie-basicfrom
auto/pie-basic/bokeh
Dec 7, 2025
Merged

feat(bokeh): implement pie-basic#365
github-actions[bot] merged 2 commits intoplot/pie-basicfrom
auto/pie-basic/bokeh

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 7, 2025

Summary

Implements pie-basic for bokeh library.

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

Implementation

  • plots/bokeh/custom/pie-basic/default.py

Changes

  • Simplified implementation to follow KISS style (no functions, no type hints)
  • Uses wedge glyphs to create pie chart (Bokeh has no native pie chart method)
  • Output size: 4800 × 2700 px per style guide
  • Features:
    • PyPlots color palette
    • Percentage labels on slices (hidden for slices < 5%)
    • Interactive tooltips with category, value, and percentage
    • Legend on right side
    • Centered title

Simplify bokeh pie chart implementation to follow KISS style:
- Remove function-based approach in favor of sequential script
- Remove type hints and docstrings
- Use 4800x2700 px output as per style guide
- Scale font sizes appropriately for high-res output
- Use PyPlots color palette
@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 e34da26 into plot/pie-basic Dec 7, 2025
8 checks passed
@github-actions github-actions bot deleted the auto/pie-basic/bokeh branch December 7, 2025 20:33
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.

1 participant