-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add boxplot params to rctemplate #5526
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
Add boxplot params to rctemplate #5526
Conversation
Don't forget to add them to the classic style, if they aren't there already. On Thu, Nov 19, 2015 at 6:24 PM, Paul Hobson notifications@github.com
|
They are already in the classic style, though I discovered a bug in them yesterday. @phobson: Maybe you could make this change here as well? diff --git a/lib/matplotlib/mpl-data/stylelib/classic.mplstyle b/lib/matplotlib/mpl-data/stylelib/classic.mplstyle
index 0f61a53..86b4c22 100644
--- a/lib/matplotlib/mpl-data/stylelib/classic.mplstyle
+++ b/lib/matplotlib/mpl-data/stylelib/classic.mplstyle
@@ -322,11 +322,11 @@ boxplot.capprops.color: k
boxplot.capprops.linestyle: -
boxplot.capprops.linewidth: 1.0
boxplot.flierprops.color: b
-boxplot.flierprops.linestyle: -
+boxplot.flierprops.linestyle: none
boxplot.flierprops.linewidth: 1.0
boxplot.flierprops.marker: +
boxplot.flierprops.markeredgecolor: k |
@mdboom done! that explains why my result images in default boxplot style PR were so o_0 |
Ok, I've confirmed that #5523 depends on this to produce meaningful test images |
MNT: Add boxplot params to rctemplate
MNT: Add boxplot params to rctemplate
back-ported to v2.0.x as 071fc5c |
Also flailing here.
Part of the work that goes with #5523.
Aims to close #5441