MRG, ENH: macOS 11 compatiblity for STC plots#8959
Conversation
fa4956e to
a2547b1
Compare
Sets `QT_MAC_WANTS_LAYER=1` for STC plots via PyVista. Was required on my system with macOS 11.2.1, Qt 5.12.9, pyqt 5.12.3 (installed from `conda-forge`) Previous fix via mne-tools#8554 didn't cover STC plots. cc @cbrnr, @GuillaumeFavelier
a2547b1 to
2aead9c
Compare
|
It's strange that conda-forge is still stuck at this old PyQt version (5.12 was released 2 years ago). Versions >=5.15.2 don't have this problem anymore. |
It seems compilation of Qt 5.15 is a blocker : |
|
@GuillaumeFavelier please merge if you think this is okay :) |
mne/viz/backends/renderer.py
Outdated
| MNE_3D_BACKEND = backend_name | ||
|
|
||
| # Qt5 macOS 11 compatibility | ||
| if (backend_name == 'pyvista' and sys.platform == 'darwin' and |
There was a problem hiding this comment.
I actually don't think the backend name matters, mayavi will have the same problem
doc/changes/latest.inc
Outdated
|
|
||
| - Reading and writing FIFF files whose filenames end with ``_meg.fif.gz``, ``_eeg.fif(.gz)``, and ``_ieeg.fif(.gz)`` doesn't emit a warning anymore; this improves interobaility with BIDS-formatted datasets (:gh:`8868` by `Richard Höchenberger`_) | ||
|
|
||
| - macOS 11 compatibility for `~mne.SourceEstimate` plots (:gh:`8959` by `Richard Höchenberger`_) |
There was a problem hiding this comment.
I think the changelog entry should explicitly mention that an env var is being created. Even if right now this only affects MNE, it can potentially affect other programs in future so we should communicate it prominently.
There was a problem hiding this comment.
My understanding is that these variables will not be visible to any other applications - only to the current Python interpreter and all subprocesses it might spawn.
There was a problem hiding this comment.
$ echo ${FOO}; python -c "import os; os.environ['FOO'] = 'bar'"; echo ${FOO}
$There was a problem hiding this comment.
Substitute "other programs" with "other Python modules" and the point makes sense though
There was a problem hiding this comment.
"other Python modules"
Oh wow, so it's not even global for everything running through the same Python interpreter? Crazy :)
There was a problem hiding this comment.
No, I meant Dan's point stands -- it will affect those
There was a problem hiding this comment.
I've improved the change log entry. Feel free to merge if you're happy!
|
Tx @hoechenberger ! |
Sets
QT_MAC_WANTS_LAYER=1for STC plots via PyVista / pyvistaqt.Was required on my system with macOS 11.2.1, Qt 5.12.9, pyqt 5.12.3 (installed from
conda-forge)Previous fix via #8554 didn't cover STC plots.
cc @cbrnr, @GuillaumeFavelier