methods matplotlib invoked raises correctly if not installed

Matplotlib is not a hard requirement to use siibra-python. It is required for a few utility functions only.

Author: AhmetNSimsek

siibra/features/tabular/receptor_density_fingerprint.py

@@ -16,7 +16,8 @@
 from .. import anchor as _anchor
 from . import tabular
 
-from ... import commons, vocabularies
+from ...commons import logger
+from ...vocabularies import RECEPTOR_SYMBOLS
 from ...retrieval import requests
 
 import pandas as pd
@@ -75,9 +76,9 @@ def neurotransmitters(self) -> List[str]:
         # Likely ill-formed tsv's
         return [
             "{} ({})".format(
-                vocabularies.RECEPTOR_SYMBOLS[t]['neurotransmitter']['label'],
-                vocabularies.RECEPTOR_SYMBOLS[t]['neurotransmitter']['name'],
-            ) if t in vocabularies.RECEPTOR_SYMBOLS else
+                RECEPTOR_SYMBOLS[t]['neurotransmitter']['label'],
+                RECEPTOR_SYMBOLS[t]['neurotransmitter']['name'],
+            ) if t in RECEPTOR_SYMBOLS else
             f"{t} (undeciphered)"
             for t in self.receptors
         ]
@@ -107,7 +108,7 @@ def parse_tsv_data(cls, data: dict):
             std = [data[_]["density (sd)"] for _ in labels]
         except KeyError as e:
             print(str(e))
-            commons.logger.error("Could not parse fingerprint from this dictionary")
+            logger.error("Could not parse fingerprint from this dictionary")
         return {
             'unit': next(iter(units)),
             'labels': labels,
@@ -124,9 +125,11 @@ def polar_plot(self, *args, backend='matplotlib', **kwargs):
         if backend == "matplotlib":
             try:
                 import matplotlib.pyplot as plt
-            except ImportError:
-                commons.logger.error("matplotlib not available. Plotting of fingerprints disabled.")
-                return None
+            except ImportError as e:
+                logger.error(
+                    "matplotlib not available. Please install matplotlib or use or another backend such as plotly."
+                )
+                raise e
             from collections import deque
 
             # default args

siibra/features/tabular/tabular.py

@@ -19,7 +19,7 @@
 
 from .. import anchor as _anchor
 
-from ... import commons
+from ...commons import logger
 
 import pandas as pd
 from textwrap import wrap
@@ -91,9 +91,11 @@ def plot(self, *args, backend="matplotlib", **kwargs):
         if backend == "matplotlib":
             try:
                 import matplotlib.pyplot as plt
-            except ImportError:
-                commons.logger.error("matplotlib not available. Plotting of fingerprints disabled.")
-                return None
+            except ImportError as e:
+                logger.error(
+                    "matplotlib not available. Please install matplotlib or use or another backend such as plotly."
+                )
+                raise e
             # default kwargs
             if kwargs.get("error_y") is None:
                 kwargs["yerr"] = kwargs.get("yerr", 'std' if 'std' in self.data.columns else None)

siibra/volumes/parcellationmap.py

@@ -703,8 +703,13 @@ def get_colormap(self, region_specs: Iterable = None):
         -------
         ListedColormap
         """
-        from matplotlib.colors import ListedColormap
-        import numpy as np
+        try:
+            from matplotlib.colors import ListedColormap
+        except ImportError as e:
+            logger.error(
+                "matplotlib not available. Please install matplotlib to create a matplotlib colormap."
+            )
+            raise e
 
         colors = {}
         if region_specs is not None: