-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Consistently use axs to refer to a set of Axes #14096
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
5dba79d
to
9bc34e8
Compare
|
||
axes[1, 0].boxplot(data, meanprops=meanpointprops, meanline=False, | ||
axs[1, 0].boxplot(data, meanprops=meanpointprops, meanline=False, |
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.
more alignment... (and below as well)
perhaps you can temporarily remove the flake8 rule excluding the check for these bad alignments to find all these cases locally :p
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.
Done. Of course that was the last missing indent - well there are still lots, but not induced by this PR.
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.
anyone can merge postci
PR Summary
Builds on top of #14079. You may wait with review until the above is in and I've rebased so that the changes are a bit smaller.Is merged.The intention of this PR is described in the addition to
pyplot.subplots()
docstring:Everything else in the commit is only applying this to examples and tests.