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.

Sign up for GitHub

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

Jump to bottom

Update boxplot default width to make it auto-adjustable #9175

Closed

richardliu-db wants to merge 2 commits into vega:main from richardliu-db:richardliu-db/boxplot-default-size

Contributor

richardliu-db commented Nov 14, 2023 •

edited

Loading

I can remove some irrelevant changes. I'm not sure if .lock file and others are expected or not.
the default bar config has the following type definition. Do we need to support ExprRef for BoxPlotConfig as well?
- defaultBarConfig: RectConfig
- interface RectConfig<ES extends ExprRef | SignalRef> extends RectBinSpacingMixins, MarkConfig
signal: "0.75 * bandwidth('x')" is just for demo purpose. Our goal is to use size: {band: 0.5}.

richardliu-db added 2 commits

November 14, 2023 13:06


          Update boxplot default size

e847bcb


          0.75 * bandwidth('x')

7dd1920

richardliu-db marked this pull request as draft

November 14, 2023 22:19

richardliu-db changed the title ~~Update boxplot default width~~ Update boxplot default width to make it auto-adjustable

richardliu-db closed this

Member

domoritz commented Feb 13, 2024

Why did you close this pull request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet