MAINT: Use MNE optional import for JAMICA

Author: snesmaeili

mne/preprocessing/ica.py

@@ -72,6 +72,7 @@
     _pl,
     _reject_data_segments,
     _require_version,
+    _soft_import,
     _validate_type,
     check_fname,
     check_random_state,
@@ -684,9 +685,6 @@ def fit(
         for method, mod in req_map.items():
             if self.method == method:
                 _require_version(mod, f"use method={repr(method)}")
-        if self.method == "jamica":
-            _require_version("jamica", "use method='jamica'", "0.3.0")
-
         _validate_type(inst, (BaseRaw, BaseEpochs), "inst", "Raw or Epochs")
 
         if np.isclose(inst.info["highpass"], 0.0):
@@ -1017,9 +1015,11 @@ def _fit(self, data, fit_type):
             self.n_iter_ = n_iter + 1  # picard() starts counting at 0
             del _, n_iter
         elif self.method == "jamica":
-            from jamica import amica
+            jamica = _soft_import(
+                "jamica", "fitting ICA with method='jamica'", min_version="0.3.0"
+            )
 
-            _, W, _, n_iter = amica(
+            _, W, _, n_iter = jamica.amica(
                 data[:, sel].T,
                 whiten=False,
                 return_n_iter=True,

mne/preprocessing/tests/test_ica.py

@@ -184,19 +184,6 @@ def test_ica_simple(method):
     assert amari_distance < 0.1
 
 
-def test_ica_jamica_missing_dependency(monkeypatch):
-    """Test the optional JAMICA dependency error."""
-    raw = RawArray(np.zeros((2, 10)), create_info(2, 100.0, "mag"), verbose=False)
-    monkeypatch.setattr(
-        "mne.utils.check.import_module", MagicMock(side_effect=ImportError)
-    )
-    with pytest.raises(
-        ImportError,
-        match=r"jamica package \(version >= 0\.3\.0\).*method='jamica'",
-    ):
-        ICA(n_components=2, method="jamica").fit(raw)
-
-
 def test_ica_jamica_multimodel_error():
     """Test that JAMICA's single-model error propagates through ICA."""
     pytest.importorskip("jamica", minversion="0.3.0")