Commit f621d4e
committed
Fix composition, npips wiring, pip_scale double-apply, tile params, domino hover
- Plotly dice + domino: axis range, reversed y, ticks, and aspect ratio now
apply on both owns-figure and caller-supplied-fig branches. Legend-reserved
x-domain [0, 0.72] / [0, 0.74] is owns-figure-only; composition uses [0, 1].
- Remove `npips` from the public API. Slot count is inferred from
`pips_order` / `pip_colors` / data — no post-preprocess overwrite.
- Collapse `tile_width` + `tile_height` into single `tile_size`. `_layout.py`
takes `tile_frac` with (0, 1] validation.
- Categorical pip radius is `layout.base_pip_r` (no double pip_scale).
`base_pip_r` already folds `pip_scale`; the backends were multiplying again.
- Plotly domino hover uses raw `size_value` via `customdata[4]` instead of the
already-scaled `marker.size`.
- Regenerated categorical showcase PNGs reflect the corrected pip size.1 parent 54f5e3b commit f621d4e
18 files changed
Lines changed: 293 additions & 95 deletions
File tree
- example_code
- images
- pydiceplot/plots
- backends
- tests
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
49 | | - | |
50 | | - | |
| 49 | + | |
51 | 50 | | |
52 | 51 | | |
53 | 52 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
36 | | - | |
| 36 | + | |
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
114 | | - | |
| 114 | + | |
115 | 115 | | |
116 | 116 | | |
117 | 117 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
51 | 51 | | |
52 | 52 | | |
53 | 53 | | |
54 | | - | |
| 54 | + | |
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
| |||
Loading
Loading
Loading
Loading
Loading
Loading
0 commit comments