Skip to content

feat(bokeh): implement box-basic#315

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

feat(bokeh): implement box-basic#315
github-actions[bot] merged 4 commits intoplot/box-basicfrom
auto/box-basic/bokeh

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Dec 6, 2025

Summary

Implements box-basic for bokeh library.

Parent Issue: #204
Sub-Issue: #233
Base Branch: plot/box-basic
Attempt: 2/3

Implementation

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

Features

  • Clean KISS-style implementation (no functions, no classes)
  • Uses spec-defined data structure with group and value columns
  • Displays Q1, median, Q3 boxes with whiskers extending to 1.5×IQR
  • Shows outliers as individual red points (#DC2626)
  • Correct figure dimensions (4800×2700 px)
  • Font sizes match default-style-guide.md recommendations (20pt labels, 16pt ticks)
  • Uses style guide color palette (#306998, #FFD43B, #DC2626, #059669)

Changes in This Attempt

  • Updated to use the official style guide color palette instead of Category10_4
  • Selenium dependencies already added for PNG export

Refactor bokeh box plot implementation to follow KISS style:
- Simple script format (no functions, no classes)
- Uses spec-defined data structure (group/value columns)
- Displays Q1, median, Q3 with whiskers extending to 1.5×IQR
- Shows outliers as individual points
- Correct figure dimensions (4800×2700 px)
- Font sizes match default-style-guide.md recommendations
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 6, 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.

- Add selenium and webdriver-manager to lib-bokeh in pyproject.toml
- Clean up box-basic implementation to KISS style

Note: CI workflow also needs Chrome setup for bokeh - to be fixed separately
Add comment to plot file to trigger CI re-run with the
selenium dependencies that were added in previous commits.
@claude claude bot closed this Dec 7, 2025
@claude claude bot reopened this Dec 7, 2025
@claude claude bot added the ai-approved Quality OK, ready for merge label Dec 7, 2025
@github-actions github-actions bot merged commit 2792833 into plot/box-basic Dec 7, 2025
13 checks passed
@github-actions github-actions bot deleted the auto/box-basic/bokeh branch December 7, 2025 10: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.

0 participants