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
capsize with default in matplotlibrc #4449
Conversation
@@ -2630,7 +2633,8 @@ def errorbar(self, x, y, yerr=None, xerr=None, | |||
The linewidth of the errorbar lines. If *None*, use the linewidth. | |||
|
|||
*capsize*: scalar | |||
The length of the error bar caps in points | |||
The length of the error bar caps in points; if *None*, it will | |||
take the value from ``errorbar.capsize`` :data:`rcParam<matplotlib.rcParams>`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is too long and hitting the pep8 checker.
Left one minor and one major comment. This looks reasonable to me other wise. Can you add an entry in https://github.com/matplotlib/matplotlib/blob/master/doc/users/whats_new/rcparams.rst please? |
The build has failed checks. I had a look at the output, but I would need some help understanding the error, I don't see how it is related to this patch. Help is appreciated. If this is some because the cap size has to be an integer, I would gladly change that everywhere, but some advice would be appreciated. |
I think the test error is unrelated. I have restarted the job |
Thanks! :) |
ENH: errorbor capsize rcparam
@HDembinski Thank you |
Some people prefer to not have ends caps on their error bars. This patch changes the default value of capsize in various function calls to "None". Inside the body of the function, "None" is then replaced by a default value taken from rcParams (the default value is still 3) under the key "errorbar.capsize". With this patch, end caps can be changed globally or turned off completely through a setting in matplotlibrc.