FIX: do not reset ylabel ha when changing position#18430
FIX: do not reset ylabel ha when changing position#18430jklymak merged 1 commit intomatplotlib:masterfrom
Conversation
jklymak
left a comment
There was a problem hiding this comment.
Looks, good, though I don't understand why you added an import 😉
| fig = Figure() | ||
| ax = fig.gca() |
There was a problem hiding this comment.
| fig = Figure() | |
| ax = fig.gca() | |
| fig, ax = plt.subplots() |
|
I was intentionally writing the tests to not invoke the pyplot machinery to see how it feels. |
It feels like you had to do an extra import for no good reason. 😉 I'm not blocking, but I'd use |
|
I'm ok with the extra import, but agree, that On a general note: What is the motivation to leave out pyplot? Is there a perfromance benefit, or is it the desire to reduce the dependency on pyplot. Is there a plan to move larger parts of the tests away from pyplot? |
708c0f3 to
95d2158
Compare
95d2158 to
ad7007b
Compare
|
Only took 4 rebases... |
PR Summary
closes #18427
This line came in via b2a80a7 / #1589 . It looks like everything still passes without it (!!) so I think that means either we never needed it or the work done since then on the colorbar code has made it redundant.
PR Checklist
pytestpasses).flake8on changed files to check).flake8-docstringsandpydocstyle<4and runflake8 --docstring-convention=all).doc/users/next_whats_new/(follow instructions in README.rst there).doc/api/next_api_changes/(follow instructions in README.rst there).