mne-tools / mne-gui-addons Goto Github PK

The PyQt import handling should be deduplicated across MNE with one abstract function at some point. This will be especially useful when PyQt versions change.

This kind of code is what needs to be deduplicated in mne.gui._ieeg_locate_gui.py

from PyQt5.QtWidgets import (QMainWindow, QGridLayout,
                             QVBoxLayout, QHBoxLayout, QLabel,
                             QMessageBox, QWidget,
                             QListView, QSlider, QPushButton,
                             QComboBox, QPlainTextEdit)

Hi,

I'm having problems correctly reading / displaying positions that were created using the mne.gui.locate_ieeg (that is the previous version, that resided in mne-python) in the current mne_gui.locate_ieeg (that is using mne-gui-addons). Although the channel positions are correctly localted when opening and displaying a fif file (containing the created info object) on my laptop with 1.4.0 mne, they look like this when opening the same file on another machine with mne dev version and mne-gui-addons pip installed from github today:

The positions were created on the 1.4.0 mne version in original CT space. The head -> CT transform is the same on both machines (identity).
Do you have an idea what could be the problem?

When marking the position of micro-channels in original CT space the locate_ieeg gui can become really slow when refreshing the main four axes after a new position in CT space was clicked. The refresh can take 3 - 5 seconds, which effectively makes the whole channel-clicking process longer (it sometimes feels that more time is spent waiting for the refreshes than actual clicking).
Do you think this could be sped up?

• Merge #3 which adds doc deployment, etc.
• Release 0.1 on GitHub, and manually release on PyPI (needs to be done once to create the package)
• Add PyPI secret for mne-gui-addons so that future releases auto-release
• Check doc deployment: https://mne.tools/mne-gui-addons/
• Get MNE-Python to use mne-gui-addons instead of built in, deprecate MNE code, add to mne sys_info: mne-tools/mne-python#11616
• Move tutorials/clinical/10_ieeg_localize.py over to this repo
• Move examples/preprocessing/locate_ieeg_micro.py over to this repo
• Add conda-forge infrastructure: conda-forge/staged-recipes#22453

Then I think the workflow will be all set!