FIX: Patch MNE internals for Pyodide/emscripten compatibility

- mne/parallel.py: return False early in _running_in_joblib_context()
  on emscripten; joblib parallel backends are unavailable in the browser
- mne/utils/config.py: catch Exception (not just ValueError) when
  loading the MNE config JSON; Pyodide's json parser raises SyntaxError
  on a corrupt or absent config file

Author: natinew77-creator

mne/parallel.py

@@ -156,6 +156,10 @@ def parallel_progress(op_iter):
 
 def _running_in_joblib_context():
     """Check if we are running in a joblib.parallel_config context manager."""
+    import sys
+
+    if sys.platform == "emscripten":
+        return False
     try:
         from joblib.parallel import get_active_backend
     except ImportError:

mne/utils/config.py

@@ -6,17 +6,19 @@
 
 import atexit
 import contextlib
+import importlib.metadata
+import importlib.util
 import json
 import multiprocessing
 import os
 import os.path as op
 import platform
 import shutil
+import site
 import subprocess
 import sys
 import tempfile
 from functools import lru_cache, partial
-from importlib import import_module
 from pathlib import Path
 from urllib.error import URLError
 from urllib.request import urlopen
@@ -272,8 +274,8 @@ def _load_config(config_path, raise_error=False):
     with _open_lock(config_path, "r+") as fid:
         try:
             config = json.load(fid)
-        except ValueError:
-            # No JSON object could be decoded --> corrupt file?
+        except Exception:
+            # Catch ANY exception (including SyntaxError from Pyodide json parser)
             msg = (
                 f"The MNE-Python config file ({config_path}) is not a valid JSON "
                 "file and might be corrupted"
@@ -757,14 +759,19 @@ def sys_info(
 
         .. versionadded:: 1.6
     """
+    import matplotlib
+
     _validate_type(dependencies, str)
     _check_option("dependencies", dependencies, ("user", "developer"))
     _validate_type(check_version, (bool, "numeric"), "check_version")
     _validate_type(unicode, (bool, str), "unicode")
     _check_option("unicode", unicode, ("auto", True, False))
     if unicode == "auto":
         if platform.system() in ("Darwin", "Linux"):
-            unicode = True
+            try:
+                unicode = sys.stdout.encoding.lower().startswith("utf")
+            except Exception:  # in case someone overrides sys.stdout in an unsafe way
+                unicode = False
         else:  # Windows
             unicode = False
     ljust = 24 if dependencies == "developer" else 21
@@ -788,6 +795,13 @@ def sys_info(
     else:
         total_memory = f"{total_memory / 1024**3:.1f}"  # convert to GiB
     out(f"{total_memory} GiB\n")
+    site_packages_path = (site.getsitepackages() or [None])[0]
+    if show_paths and site_packages_path is not None:
+        out("Site-packages".ljust(ljust) + f"{site_packages_path}\n")
+        site_packages_path = Path(site_packages_path)
+        out("".ljust(ljust))
+        out("└►" if unicode else "^-")
+        out(" Any paths not listed below are in site-packages")
     out("\n")
     ljust -= 3  # account for +/- symbols
     libs = _get_numpy_libs()
@@ -800,12 +814,13 @@ def sys_info(
         "matplotlib",
         "",
         "# Numerical (optional)",
-        "sklearn",
+        "scikit-learn",
         "numba",
         "nibabel",
         "nilearn",
         "dipy",
         "openmeeg",
+        "python-picard",
         "cupy",
         "pandas",
         "h5io",
@@ -820,7 +835,7 @@ def sys_info(
         "pyqtgraph",
         "mne-qt-browser",
         "ipywidgets",
-        # "trame",  # no version, see https://github.com/Kitware/trame/issues/183
+        "trame",
         "trame_client",
         "trame_server",
         "trame_pyvista",
@@ -834,6 +849,7 @@ def sys_info(
         "mne-connectivity",
         "mne-icalabel",
         "mne-bids-pipeline",
+        "autoreject",
         "neo",
         "eeglabio",
         "edfio",
@@ -849,6 +865,18 @@ def sys_info(
         use_mod_names += (
             "# Testing",
             "pytest",
+            "pytest-cov",
+            "pytest-qt",
+            "pytest-rerunfailures",
+            "pytest-timeout",
+            "codespell",
+            "ipython",
+            "mypy",
+            "pillow",
+            "pre-commit",
+            "ruff",
+            "vulture",
+            "",
             "hedtools",
             "statsmodels",
             "numpydoc",
@@ -859,6 +887,7 @@ def sys_info(
             "imageio",
             "imageio-ffmpeg",
             "snirf",
+            "twine",
             "",
             "# Documentation",
             "sphinx",
@@ -874,12 +903,24 @@ def sys_info(
             "tqdm",
             "",
         )
-    try:
-        unicode = unicode and (sys.stdout.encoding.lower().startswith("utf"))
-    except Exception:  # in case someone overrides sys.stdout in an unsafe way
-        unicode = False
-    mne_version_good = True
-    import_names = dict(hedtools="hed")
+    if check_version:
+        timeout = 2.0 if check_version is True else float(check_version)
+        mne_version_good, mne_extra = _check_mne_version(timeout)
+        if mne_version_good is None:
+            mne_version_good = True
+        del timeout
+    else:
+        mne_version_good = True
+        mne_extra = ""
+    del check_version
+    import_names = {
+        "codespell": "codespell_lib",
+        "hedtools": "hed",
+        "ipython": "IPython",
+        "pillow": "PIL",
+        "pytest-qt": "pytestqt",
+        "scikit-learn": "sklearn",
+    }
     for mi, mod_name in enumerate(use_mod_names):
         # upcoming break
         if mod_name == "":  # break
@@ -897,62 +938,50 @@ def sys_info(
             continue
         pre = "├"
         last = use_mod_names[mi + 1] == "" and not unavailable
+        import_name = import_names.get(mod_name, mod_name).replace("-", "_")
         if last:
             pre = "└"
         try:
-            import_name = import_names.get(mod_name, mod_name.replace("-", "_"))
-            mod = import_module(import_name)
+            ver = importlib.metadata.version(mod_name)
+            mod_loc = Path(importlib.util.find_spec(import_name).origin)
         except Exception:
             unavailable.append(mod_name)
         else:
+            if mod_loc.stem == "__init__":
+                mod_loc = mod_loc.parent
+            if site_packages_path and mod_loc.is_relative_to(site_packages_path):
+                mod_loc = None
             mark = "☑" if unicode else "+"
-            mne_extra = ""
-            if mod_name == "mne" and check_version:
-                timeout = 2.0 if check_version is True else float(check_version)
-                mne_version_good, mne_extra = _check_mne_version(timeout)
-                if mne_version_good is None:
-                    mne_version_good = True
-                elif not mne_version_good:
-                    mark = "☒" if unicode else "X"
+            if mod_name == "mne" and not mne_version_good:
+                mark = "☒" if unicode else "X"
             out(f"{pre}{mark} " if unicode else f" {mark} ")
             out(f"{mod_name}".ljust(ljust))
-            if mod_name == "vtk":
-                vtk_version = mod.vtkVersion()
-                # 9.0 dev has VersionFull but 9.0 doesn't
-                for attr in ("GetVTKVersionFull", "GetVTKVersion"):
-                    if hasattr(vtk_version, attr):
-                        version = getattr(vtk_version, attr)()
-                        if version != "":
-                            out(version)
-                            break
-                else:
-                    out("unknown")
-            else:
-                out(mod.__version__.lstrip("v"))
+            out(ver)
             if mod_name == "numpy":
                 out(f" ({libs})")
             elif mod_name == "qtpy":
                 version, api = _check_qt_version(return_api=True)
                 out(f" ({api}={version})")
             elif mod_name == "matplotlib":
-                out(f" (backend={mod.get_backend()})")
+                out(f" (backend={matplotlib.get_backend()})")
             elif mod_name == "pyvista":
                 version, renderer = _get_gpu_info()
                 if version is None:
                     out(" (OpenGL unavailable)")
                 else:
                     out(f" (OpenGL {version} via {renderer})")
             elif mod_name == "mne":
-                out(f" ({mne_extra})")
+                if mne_extra:
+                    out(f" ({mne_extra})")
             # Now comes stuff after the version
-            if show_paths:
+            if show_paths and mod_loc is not None:
                 if last:
                     pre = "   "
                 elif unicode:
                     pre = "│  "
                 else:
                     pre = " | "
-                out(f"\n{pre}{' ' * ljust}{op.dirname(mod.__file__)}")
+                out(f"\n{pre}{' ' * ljust}{mod_loc}")
             out("\n")
 
     if not mne_version_good:
@@ -986,7 +1015,7 @@ def _check_mne_version(timeout):
     if not rel_ver[0].isnumeric():
         return None, (f"unable to check for latest version on GitHub, {rel_ver}")
     rel_ver = parse(rel_ver)
-    this_ver = parse(import_module("mne").__version__)
+    this_ver = parse(importlib.metadata.version("mne"))
     if this_ver > rel_ver:
         return True, f"development, latest release is {rel_ver}"
     if this_ver == rel_ver: