DOC: modernise barh example #30973
DOC: modernise barh example #30973
Conversation
|
|
||
| ax.barh(people, performance, xerr=error, align='center') | ||
| ax.invert_yaxis() # labels read top-to-bottom | ||
| ax.yaxis.set_inverted(True) # labels read top-to-bottom |
There was a problem hiding this comment.
This is preferred to ax.invert_yaxis ? Not sure I understand that.
There was a problem hiding this comment.
See https://github.com/matplotlib/matplotlib/pull/13330/changes#diff-23b7fa7a4b346c0e303ac8bc2158c701c855dc79256dff8bfadbaa9df62307e5: the problem of invert_yaxis is that it is brittle if you don't control the whole axes life-cycle, which easily occurs if your code ends up refactored in a separate function.
There was a problem hiding this comment.
"invert" means "to reverse in position, order, or relationship". It doesn't matter if the axis has been inverted before, it's just a toggle, so I don't see the brittleness
fig, ax = plt.subplots()
ax.plot(np.arange(10))
ax.invert_yaxis()
ax.invert_yaxis()
yields a yaxis that has 0 on the bottom and 9 on the top
|
I think this kind of minimal example should just use some made-up data ( |
|
@rcomer thanks for immediately taking action on adding fixed data. |
Co-authored-by: Tim Hoffmann <2836374+timhoffm@users.noreply.github.com>
PR summary