-
Notifications
You must be signed in to change notification settings - Fork 2k
Add support for per-category width on boxplot. #1962
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1962 +/- ##
=========================================
Coverage ? 95.16%
=========================================
Files ? 12
Lines ? 4468
Branches ? 0
=========================================
Hits ? 4252
Misses ? 216
Partials ? 0
Continue to review full report at Codecov.
|
I think that if this were going to be supported, it would have to be similar to how that option works in violinplot. |
Thanks, you mean in terms of preserving the area? Or in terms of
automatically using the number of samples to determine the relative width?
…On Fri, Feb 14, 2020, 4:37 PM Michael Waskom ***@***.***> wrote:
I think that if this were going to be supported, it would have to be
similar to how that option works in violinplot.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1962?email_source=notifications&email_token=AAJGIIGNHOFRDGGC53I74UDRC42M5A5CNFSM4KVRIXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL24MGI#issuecomment-586532377>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJGIICHD4IC62P3OJ7O4N3RC42M5ANCNFSM4KVRIXFA>
.
|
I see, do you mean using the scale parameter instead and preserve the
semantics of weight as they are?
If so that makes sense, let me try it.
…On Fri, Feb 14, 2020, 5:51 PM Elmer ***@***.***> wrote:
Thanks, you mean in terms of preserving the area? Or in terms of
automatically using the number of samples to determine the relative width?
On Fri, Feb 14, 2020, 4:37 PM Michael Waskom ***@***.***>
wrote:
> I think that if this were going to be supported, it would have to be
> similar to how that option works in violinplot.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#1962?email_source=notifications&email_token=AAJGIIGNHOFRDGGC53I74UDRC42M5A5CNFSM4KVRIXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL24MGI#issuecomment-586532377>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAJGIICHD4IC62P3OJ7O4N3RC42M5ANCNFSM4KVRIXFA>
> .
>
|
This is now obviated (without explicit effort to enable it) by #3294: sns.boxplot(data=tips, x="day", y="total_bill", widths=[.2, .4, .6, .8]) It doesn't work with Thanks for your interest! |
Support per-category width on boxplot. As discused on #1668, this is done by passing an array of widths on the sns.boxplot(width) argument.
Please let me know if this makes sense. I can add further testing.