generate list of path effects in RcParams object

story645 · story645 · commit 06d8b29a107f · 2023-09-22T00:23:27.000-04:00
diff --git a/lib/matplotlib/__init__.py b/lib/matplotlib/__init__.py
@@ -163,7 +163,6 @@
 from matplotlib._api import MatplotlibDeprecationWarning
 from matplotlib.rcsetup import validate_backend, cycler
 
-
 _log = logging.getLogger(__name__)
 
 __bibtex__ = r"""@Article{Hunter:2007,
@@ -809,6 +808,21 @@ def copy(self):
             rccopy._set(k, self._get(k))
         return rccopy
 
+    def _load_path_effects(self):
+        """defers loading of patheffects to avoid circular imports"""
+        import matplotlib.patheffects as path_effects
+
+        path_effects_list = self._get('path.effects').copy()
+        print("before_loop: ", path_effects_list)
+        for k, v in self.find_all('path.effects.').items():
+            if v is None:
+                continue
+            path_effect_name = k.split('.')[-1]
+            path_effect_function = getattr(path_effects, path_effect_name)
+            path_effects_list.append(path_effect_function(**v))
+        print("after_loop: ",  path_effects_list)
+        return path_effects_list
+
 
 def rc_params(fail_on_error=False):
     """Construct a `RcParams` instance from the default Matplotlib rc file."""
diff --git a/lib/matplotlib/artist.py b/lib/matplotlib/artist.py
@@ -209,7 +209,7 @@ def __init__(self):
         self._gid = None
         self._snap = None
         self._sketch = mpl.rcParams['path.sketch']
-        self._path_effects = []
+        self._path_effects = mpl.rcParams._load_path_effects()
         self._sticky_edges = _XYPair([], [])
         self._in_layout = True
 
diff --git a/lib/matplotlib/patheffects.py b/lib/matplotlib/patheffects.py
@@ -5,11 +5,8 @@
 .. seealso::
    :ref:`patheffects_guide`
 """
-import sys
 
-import matplotlib as mpl
 from matplotlib.backend_bases import RendererBase
-import matplotlib.cbook as cbook
 from matplotlib import colors as mcolors
 from matplotlib import patches as mpatches
 from matplotlib import transforms as mtransforms
@@ -93,13 +90,7 @@ def __init__(self, path_effects, renderer):
         renderer : `~matplotlib.backend_bases.RendererBase` subclass
 
         """
-        compound_path_effects = mpl.rcParams['path.effects']
-        for k, v in mpl.rcParams.find_all('path.effects.*').copy().items():
-            if path_effect_name := k.lstrip('path.effects') and v is not None:
-                path_effect_function = sys.modules[__name__].__dict__[path_effect_name]
-                compound_path_effects.append(path_effect_function(**v))
-        compound_path_effects.append(path_effects)
-        self._path_effects = cbook.flatten(compound_path_effects)
+        self._path_effects = path_effects
         self._renderer = renderer
 
     def copy_with_path_effect(self, path_effects):
