feat(plotly): add reactive strip chart selection support by axsseldz · Pull Request #9012 · marimo-team/marimo

axsseldz · 2026-04-03T12:55:17Z

📝 Summary

Makes px.strip fully selection-reactive in mo.ui.plotly. Strip charts are go.Box traces under the hood (boxpoints="all", transparent fill), so this PR combines the box and violin implementations from the sibling branches onto one branch.

🔍 Description of Changes

Frontend: added type === "box" and type === "violin" to shouldHandleClickSelection.
Python: ported the full box and violin implementations (_append_*, _extract_*, _build_* helpers). Strip chart reactivity comes for free through the box path.
Tests: violin tests.
Example: examples/third_party/plotly/strip_chart.py — px.strip() grouped, go.Box single-trace, and horizontal variants with live stats and mo.ui.table.

Box Selection

Click Selection

📋 Pre-Review Checklist

For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
Any AI generated code has been reviewed line-by-line by the human PR author, who stands by it.
Video or media evidence is provided for any visual changes (optional).

✅ Merge Checklist

I have read the contributor guidelines.
Documentation has been updated where applicable, including docstrings for API changes.
Tests have been added for the changes made.

@mscolnick @nojaf

vercel · 2026-04-03T12:55:22Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
marimo-docs	Ready	Preview, Comment	Apr 9, 2026 1:41am

Copilot

Pull request overview

Enables selection reactivity for Plotly box/violin traces (and therefore px.strip, which renders as go.Box) in mo.ui.plotly, including click-selection support in the frontend and row-level expansion/extraction on the Python side.

Changes:

Frontend: handle click selections for type === "box" and type === "violin".
Python: add box/violin selection expansion/extraction helpers and hook them into _convert_value.
Add extensive tests plus a new strip-chart example and docs update.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
`marimo/_plugins/ui/_impl/plotly.py`	Adds box/violin selection expansion (click `pointNumbers`) and range-based extraction logic.
`tests/_plugins/ui/_impl/test_plotly.py`	Adds unit tests covering box/violin/strip selection conversion and extraction helpers.
`frontend/src/plugins/impl/plotly/selection.ts`	Allows click-selection handling for box/violin traces.
`frontend/src/plugins/impl/plotly/__tests__/selection.test.ts`	Adds tests asserting click-selection eligibility for box/violin.
`examples/third_party/plotly/strip_chart.py`	New example demonstrating reactive strip-chart selection workflows.
`docs/api/plotting.md`	Updates documentation of which Plotly chart types support reactive selection.

marimo/_plugins/ui/_impl/plotly.py

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

marimo/_plugins/ui/_impl/plotly.py

Resolves conflicts between waterfall (merged to main) and strip/box/violin additions. Both feature sets are now present: waterfall handlers are inserted before the box/violin handlers in _convert_value, and all function definitions are included. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Resolves conflicts between waterfall (merged to main) and funnel additions. Both feature sets included: waterfall handlers inserted before funnel handlers in _convert_value, all function definitions and tests from both branches kept. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Fix unsafe np.asarray(val_data, dtype=np.float64) call: wrap in try/except and fall back to element-wise extraction when val_data contains None or non-numeric entries (comment 1) - Fix off-by-one in overlap check: use >= val_min (inclusive lower bound) in both the numpy fast path and the pure-Python fallback, consistent with bar selection behavior (comment 2) - Correct docstring: click path strips empty placeholders and re-syncs indices; it does not deduplicate (comment 3) - Update docs to mention funnelarea support explicitly (comment 4) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Merges violin plot and area chart support from main while preserving box/strip plot additions. `hasAreaTrace` added to TS, both `_append_box_points_to_selection` and `_append_violin_points_to_selection` coexist in Python with their full helper function hierarchies. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Syncs strip chart support into box branch so both can merge into main independently without conflicts. Adds strip chart example, strip test section, and updated supported-charts documentation. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Adds funnel/funnelarea chart support from feat/reactive-plotly-funnel so that both branches can merge into main independently without conflicts. Includes funnel/funnelarea trace types, test cases, and docs updates. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The cross-branch merge left two competing implementations of _append_box_points_to_selection and its helpers, plus duplicate _build_waterfall_point and _append_map_scatter_points_to_selection. Python silently used the last definition; pyright caught all six [no-redef] errors in CI. Removes the dead first copies. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

frontend/src/plugins/impl/plotly/selection.ts

examples/third_party/plotly/strip_chart.py

github-actions bot added the documentation Improvements or additions to documentation label Apr 3, 2026

vercel bot deployed to Preview April 3, 2026 12:56 View deployment

axsseldz marked this pull request as ready for review April 3, 2026 13:57

axsseldz requested review from Light2Dark, akshayka and manzt as code owners April 3, 2026 13:57

Copilot AI review requested due to automatic review settings April 3, 2026 13:57

Copilot started reviewing on behalf of axsseldz April 3, 2026 13:58 View session

Copilot AI reviewed Apr 3, 2026

View reviewed changes

mscolnick requested review from kirangadhave and removed request for Light2Dark, akshayka and manzt April 3, 2026 18:12

feat(plotly): add reactive strip selection support

d4d23d8

axsseldz force-pushed the feat/reactive-plotly-strip branch from b2adcbe to a9d7b60 Compare April 3, 2026 23:58

vercel bot deployed to Preview April 3, 2026 23:59 View deployment

axsseldz force-pushed the feat/reactive-plotly-strip branch from a9d7b60 to 75c505f Compare April 4, 2026 00:18

vercel bot deployed to Preview April 4, 2026 00:19 View deployment

axsseldz requested a review from Copilot April 4, 2026 01:11

Copilot started reviewing on behalf of axsseldz April 4, 2026 01:12 View session

Copilot AI reviewed Apr 4, 2026

View reviewed changes

marimo/_plugins/ui/_impl/plotly.py Outdated Show resolved Hide resolved

marimo/_plugins/ui/_impl/plotly.py Outdated Show resolved Hide resolved

Copilot comments resolved

0891779

axsseldz force-pushed the feat/reactive-plotly-strip branch from 75c505f to 0891779 Compare April 4, 2026 02:10

vercel bot deployed to Preview April 4, 2026 02:12 View deployment

axsseldz and others added 5 commits April 3, 2026 20:01

feat(plotly): add reactive funnel selection support

328a547

feat(plotly): add reactive box selection support

ed2da21

Copilot comments resolved

d3852fa

axsseldz and others added 3 commits April 8, 2026 01:31

refactor(plotly): reduce type assertions in selection.ts

12e7cef

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

refactor(plotly): reduce unnecessary type assertions in selection.ts

dace8ee

vercel bot deployed to Preview April 8, 2026 08:45 View deployment

axsseldz added 2 commits April 8, 2026 01:47

refactor(plotly): reduce unnecessary type assertions in selection.ts

99219ff

fix(plotly): fix malformed test structure in selection.test.ts

fead98e

axsseldz mentioned this pull request Apr 8, 2026

feat(plotly): add reactive funnel and funnelarea selection support #9044

Open

6 tasks

axsseldz and others added 12 commits April 8, 2026 09:02

fix(plotly): remove duplicate violin type check in strip selection.ts

9b23e9e

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Merge main into feat/reactive-plotly-box

014aa72

fix(plotly): repair malformed test file in box branch merge

879c5e9

Merge main into feat/reactive-plotly-funnel

0f2f405

fix(plotly): remove duplicate violin check in strip _convert_value

f3e1c0c

fix(plotly): remove duplicate violin tests in strip test file

73c70c9

merge feat/reactive-plotly-funnel into plotly-box for cross-branch sync

3020767

fix(plotly): remove duplicate box/violin test cases in selection.test.ts

4d2ec4c

fix(plotly): remove duplicate box/violin test cases in selection.test.ts

fac08d2

Light2Dark mentioned this pull request Apr 8, 2026

feat(plotly): add reactive box plot selection support #9010

Open

6 tasks

vercel bot deployed to Preview April 8, 2026 17:14 View deployment

vercel bot deployed to Preview April 8, 2026 17:26 View deployment

mscolnick requested a review from Copilot April 8, 2026 19:43

Copilot started reviewing on behalf of mscolnick April 8, 2026 19:44 View session

Copilot AI reviewed Apr 8, 2026

View reviewed changes

frontend/src/plugins/impl/plotly/selection.ts Show resolved Hide resolved

examples/third_party/plotly/strip_chart.py Outdated Show resolved Hide resolved

Merge branch 'feat/reactive-plotly-box' into feat/reactive-plotly-strip

64bd761

axsseldz force-pushed the feat/reactive-plotly-strip branch from 3c2b51d to 64bd761 Compare April 9, 2026 01:40

vercel bot deployed to Preview April 9, 2026 01:41 View deployment

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(plotly): add reactive strip chart selection support#9012

feat(plotly): add reactive strip chart selection support#9012
axsseldz wants to merge 26 commits intomarimo-team:mainfrom
axsseldz:feat/reactive-plotly-strip

axsseldz commented Apr 3, 2026 •

edited

Loading

Uh oh!

vercel bot commented Apr 3, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

axsseldz commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📝 Summary

🔍 Description of Changes

Box Selection

Click Selection

📋 Pre-Review Checklist

✅ Merge Checklist

Uh oh!

vercel bot commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

axsseldz commented Apr 3, 2026 •

edited

Loading

vercel bot commented Apr 3, 2026 •

edited

Loading