WIP: Port 3d testing to BackgroundPlotter completely#7697
WIP: Port 3d testing to BackgroundPlotter completely#7697GuillaumeFavelier wants to merge 6 commits intomne-tools:masterfrom
Conversation
|
I reported this behaviour in https://github.com/pyvista/pyvista/issues/703 |
|
One way to mitigate this issue is to use |
The drawback is that the user is no longer able to resize the window to a geometry smaller then |
|
This can be relevant to #7211 (about minimum/fixed widget geometry) |
Can you set this until the |
|
I use it after |
Can you see if it's possible to do before initializing the view? It should help prevent more jerky resizes, if those happen with the current approach |
|
The window is resized before being shown so I don't think this will lead to jerkiness. But this is a workaround without a way to setting it at init. So far, locally the main issue I notice is with I have to manually resize to get all the content: Also, the CIs do not seem to like those changes. |
|
Probably whatever |
|
For now you could |
|
Awesome idea, thank you. I was stuck with |
|
The Azure failure is fixed by #7740, but I couldn't see the failed OSX log so I restarted that build. |
|
Failures on OSX apparently seem real https://travis-ci.org/github/mne-tools/mne-python/builds/683971748 |
|
Actually if #7758 is chosen, I would prefer to keep testing on both plotters. |


This PR ports testing of the PyVista 3d backend to the
BackgroundPlottercompletely.As a consequence, the variable
MNE_3D_BACKEND_TESTINGneeds refactoring.Known issue
BackgroundPlotterare not equal towin_height x win_width x 3but slightly less. I suppose that the real dimensions taken into account are effectively those of the render view (which is smaller than the window).