Commit d08bd6e
feat(bokeh): implement histogram-basic (#309)
## Summary
Implements `histogram-basic` for **bokeh** library.
**Parent Issue:** #206
**Sub-Issue:** #228
**Base Branch:** `plot/histogram-basic`
**Attempt:** 3/3
## Implementation
- `plots/bokeh/quad/histogram-basic/default.py`
## Changes
1. Basic histogram implementation using bokeh's `quad` glyph
2. Uses numpy to compute histogram bins (30 bins for good visual
distribution)
3. Follows style guide with 4800x2700 px output dimensions
## Dependencies
- `selenium>=4.15.0` and `webdriver-manager>=4.0.0` are already in
`lib-bokeh` dependencies in `pyproject.toml` (required for `export_png`)
## CI Notes
**Important:** The CI workflow needs to be updated to set up Chrome for
bokeh PNG export (similar to highcharts).
The fix is to update `.github/workflows/ci-plottest.yml`:
```yaml
- name: Setup Chrome for Highcharts/Bokeh
if: steps.detect_libs.outputs.has_plots == 'true' && (contains(steps.detect_libs.outputs.libraries, 'highcharts') || contains(steps.detect_libs.outputs.libraries, 'bokeh'))
uses: browser-actions/setup-chrome@v1
with:
chrome-version: stable
```
## Verified Locally
- Implementation tested locally and generates correct plot.png
(4800x2700 px)
- Code passes ruff format and lint checks
---------
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Markus Neusinger <2921697+MarkusNeusinger@users.noreply.github.com>1 parent 4ad896b commit d08bd6e
1 file changed
Lines changed: 51 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
0 commit comments