Add a draw during show for macos backend #27955
Conversation
Without this a stale figure is shown
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
|
Ping for second review here, I'd like to get this in for 3.8.4, which I would like to get out soon for np2 reasons. I'm not too concerned about potentially doubling up draw_idle requests here, as they are intentionally pretty cheap in the first place. I am more interested in getting the fix in than tracking down if other calls could possibly be omitted. I think it is likely correct that we could omit the call in |
…955-on-v3.8.x Backport PR #27955 on branch v3.8.x (Add a draw during show for macos backend)
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