Skip to content

Add box mark to auto #1423

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Add box mark to auto #1423

wants to merge 4 commits into from

Conversation

tophtucker
Copy link
Contributor

@tophtucker tophtucker commented Apr 4, 2023

Plot.auto would never choose box by default, but you could override to it, thereby allowing box plots in the chart cell on Observable. See #1422

@tophtucker
Copy link
Contributor Author

Still a decent idea, at least for box plots, but stale and trivial, so closing for now! More in the issue: #1422

@tophtucker tophtucker closed this Aug 24, 2023
@mbostock
Copy link
Member

Seems like a quick win if we want to add box charts to the chart cell. I’m supportive! (Voronoi seems more dubious.)

@tophtucker tophtucker reopened this Aug 24, 2023
@tophtucker tophtucker changed the title Sketch: adding override-only mark types to auto Add box mark to auto Aug 24, 2023
@tophtucker
Copy link
Contributor Author

K, reviving! 🧟 Box only now. I wonder if we should try some approach for the quant-quant case like you describe in #1330 (comment). I wanna test a bit more… notebook demo: https://observablehq.com/d/5efa14f16d261488

@tophtucker
Copy link
Contributor Author

More test cases in notebook now: https://observablehq.com/d/5efa14f16d261488

Breaks if color is anything but redundantly encoding categorical dimension:

image

Cat-cat shows nothing:

image

Quant-quant is tricky (see #1330):

image

@tophtucker
Copy link
Contributor Author

Just checking in on my stale PRs…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants