Add a draw during show for macos backend - #27955
Conversation
Without this a stale figure is shown
greglucas
left a comment
greglucas
left a comment
There was a problem hiding this comment.
Thanks for the pointer about this needing to be run in a terminal and not a script. I can reproduce the original issue that way.
This fixes the issue, so has my approval and is good to go in. But I am still not sure whether there is a deeper issue here with the way we are setting the interrupt handling that this is just covering up in a sense by forcing another event through.
In the __init__ method, we are actually following the call to self.show() with a draw_idle(). I'm wondering if we can either remove that call now, or if we should just be calling draw_idle() here regardless of whether we are stale or not like above.
matplotlib/lib/matplotlib/backends/backend_macosx.py
Lines 159 to 161 in 1174aad
…955-on-v3.8.x Backport PR #27955 on branch v3.8.x (Add a draw during show for macos backend)
4 participants
Without this a stale figure is shown
PR summary
This is basically the same concept as what happens just before show in
plt.pause.Essentially, there is nothing in the macos backend that actually causes a stale figure to be redrawn on
show.Other backends (qt, tk in particular, but likely others) seem to be saved by the fact that a resize event is triggered upon show, but macos does not do the same.
Other ideas considered, would be to do this in
plt.show(which would perhaps leave out some other mechanisms, such as mpl-gui) or to try and do this in backend_bases (which is hard because most do not callsuper).Could also leave out the condition and just always call
draw_idle.closes #27953
PR checklist