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
boxplot/violinplot orientation-setting API #13435
Comments
I think this is a good idea. Should be easy enough to capture |
(and while we're at it, the manage_xticks arg should be renamed to manage_ticks, as it covers both orientations) |
just throwing this out there: what would you say to |
well technically it's managing ticks, tick labels, and axis limits so I think manage_labels is worse than manage_ticks, and perhaps manage_axis is better? |
good points all around. just noticed a wrinkle: |
We know how to handle deprecation/renaming of rcParams too, but thanks for pointing that out. |
This issue has been marked "inactive" because it has been 365 days since the last comment. If this issue is still present in recent Matplotlib releases, or the feature request is still wanted, please leave a comment and this label will be removed. If there are no updates in another 30 days, this issue will be automatically closed, but you are free to re-open or create a new issue if needed. We value issue reports, and this procedure is meant to help us resurface and prioritize issues that have not been addressed yet, not make them disappear. Thanks for your help! |
At least adding orientation should hopefully be uncontroversial. |
Would you want separate PRs for |
Should we simply deprecate the I believe, we also don’t have that config for |
I think that's a good idea and the justification is solid, IMO |
Bug report
Bug summary
Currently, boxplot(), bxp(), violin(), and violinplot() take a
vert: bool
argument to switch between vertical and horizontal; this is inconsistent with colorbar(), hist(), eventplot(), and Slider(), which take anorientation: {"vertical", "horizontal"}
argument.For consistency, I'd suggest adding support for
orientation
(both because I think colorbar() and hist(), in particular, are probably the most used among all these, and because I like the API better :-)), and later deprecatevert
. Thoughts?attn @phobson who has been involved in boxplots, IIRC.
The text was updated successfully, but these errors were encountered: