maidr 1.2.0__py3-none-any.whl → 1.2.2__py3-none-any.whl

maidr/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "1.2.0"
+__version__ = "1.2.2"
 
 from .api import close, render, save_html, show, stacked
 from .core import Maidr

maidr/core/plot/lineplot.py

@@ -8,6 +8,7 @@ from maidr.core.enum.plot_type import PlotType
 from maidr.core.plot.maidr_plot import MaidrPlot
 from maidr.exception.extraction_error import ExtractionError
 from maidr.util.mixin.extractor_mixin import LineExtractorMixin
+import uuid
 
 
 class MultiLinePlot(MaidrPlot, LineExtractorMixin):
@@ -42,62 +43,75 @@ class MultiLinePlot(MaidrPlot, LineExtractorMixin):
         super().__init__(ax, PlotType.LINE)
 
     def _get_selector(self) -> Union[str, List[str]]:
-        return ["g[maidr='true'] > path"]
+        # Return selectors for all lines that have data
+        all_lines = self.ax.get_lines()
+        if not all_lines:
+            return ["g[maidr='true'] > path"]
+
+        selectors = []
+        for line in all_lines:
+            # Only create selectors for lines that have data (same logic as _extract_line_data)
+            xydata = line.get_xydata()
+            if xydata is None or not xydata.size:  # type: ignore
+                continue
+            gid = line.get_gid()
+            if gid:
+                selectors.append(f"g[id='{gid}'] path")
+            else:
+                selectors.append("g[maidr='true'] > path")
+
+        if not selectors:
+            return ["g[maidr='true'] > path"]
+
+        return selectors
 
-    def _extract_plot_data(self) -> list[dict]:
-        plot = self.extract_lines(self.ax)
-        data = self._extract_line_data(plot)
+    def _extract_plot_data(self) -> Union[List[List[dict]], None]:
+        data = self._extract_line_data()
 
         if data is None:
-            raise ExtractionError(self.type, plot)
+            raise ExtractionError(self.type, None)
 
         return data
 
-    def _extract_line_data(
-        self, plot: Union[List[Line2D], None]
-    ) -> Union[List[dict], None]:
+    def _extract_line_data(self) -> Union[List[List[dict]], None]:
         """
-        Extract data from multiple line objects.
-
-        Parameters
-        ----------
-        plot : list[Line2D] | None
-            List of Line2D objects to extract data from.
+        Extract data from all line objects and return as separate arrays.
 
         Returns
         -------
-        list[dict] | None
-            List of dictionaries containing x,y coordinates and line identifiers,
-            or None if the plot data is invalid.
+        list[list[dict]] | None
+            List of lists, where each inner list contains dictionaries with x,y coordinates
+            and line identifiers for one line, or None if the plot data is invalid.
         """
-        if plot is None or len(plot) == 0:
+        all_lines = self.ax.get_lines()
+        if not all_lines:
             return None
 
-        all_line_data = []
+        all_lines_data = []
 
-        # Process each line in the plot
-        for i, line in enumerate(plot):
-            if line.get_xydata() is None:
+        for line in all_lines:
+            xydata = line.get_xydata()
+            if xydata is None or not xydata.size:  # type: ignore
                 continue
 
-            # Tag the element for highlighting
             self._elements.append(line)
 
-            # Extract data from this line
+            # Assign unique GID to each line if not already set
+            if line.get_gid() is None:
+                unique_gid = f"maidr-{uuid.uuid4()}"
+                line.set_gid(unique_gid)
 
             label: str = line.get_label()  # type: ignore
             line_data = [
                 {
                     MaidrKey.X: float(x),
                     MaidrKey.Y: float(y),
-                    # Replace labels starting with '_child'
-                    # with an empty string to exclude
-                    # internal or non-relevant labels from being used as identifiers.
                     MaidrKey.FILL: (label if not label.startswith("_child") else ""),
                 }
                 for x, y in line.get_xydata()  # type: ignore
             ]
-            if len(line_data) > 0:
-                all_line_data.append(line_data)
 
-        return all_line_data if all_line_data else None
+            if line_data:
+                all_lines_data.append(line_data)
+
+        return all_lines_data if all_lines_data else None

maidr/patch/lineplot.py

@@ -7,29 +7,39 @@ from matplotlib.lines import Line2D
 from maidr.core.enum import PlotType
 from maidr.patch.common import common
 from maidr.core.enum.smooth_keywords import SMOOTH_KEYWORDS
+from maidr.core.context_manager import ContextManager
+from maidr.core.figure_manager import FigureManager
 
 
 def line(wrapped, instance, args, kwargs) -> Axes | list[Line2D]:
     """
-    Wrapper function for line plotting functions in matplotlib and seaborn.
-
-    Parameters
-    ----------
-    wrapped : callable
-        The wrapped function (plot or lineplot)
-    instance : object
-        The object to which the wrapped function belongs
-    args : tuple
-        Positional arguments passed to the wrapped function
-    kwargs : dict
-        Keyword arguments passed to the wrapped function
-
-    Returns
-    -------
-    Axes | list[Line2D]
-        The result of the wrapped function after processing
+    Wrapper for line plotting functions that creates a single MAIDR plot per axes to handle
+    multiline plots (matplotlib) and single-call plots (seaborn) correctly by preventing
+    multiple MAIDR layers and using internal context to avoid cyclic processing.
     """
-    return common(PlotType.LINE, wrapped, instance, args, kwargs)
+    # Don't proceed if the call is made internally by the patched function.
+    if ContextManager.is_internal_context():
+        return wrapped(*args, **kwargs)
+
+    # Set the internal context to avoid cyclic processing.
+    with ContextManager.set_internal_context():
+        # Patch the plotting function.
+        plot = wrapped(*args, **kwargs)
+
+    # Get the axes from the plot result (works for both matplotlib and seaborn)
+    ax = FigureManager.get_axes(plot)
+    if ax is None:
+        # If we can't get axes from plot, try from instance
+        ax = instance if isinstance(instance, Axes) else getattr(instance, "axes", None)
+
+    # Check if a MAIDR plot already exists for this axes
+    if ax is not None and not hasattr(ax, "_maidr_plot_created"):
+        # Create MAIDR plot only once for this axes using common()
+        common(PlotType.LINE, lambda *a, **k: plot, instance, args, kwargs)
+        # Mark that a MAIDR plot has been created for this axes
+        setattr(ax, "_maidr_plot_created", True)
+
+    return plot
 
 
 # Patch matplotlib function.

{maidr-1.2.0.dist-info → maidr-1.2.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: maidr
-Version: 1.2.0
+Version: 1.2.2
 Summary: Multimodal Access and Interactive Data Representations
 License: GPL-3.0-or-later
 Keywords: accessibility,visualization,sonification,braille,tactile,multimodal,data representation,blind,low vision,visual impairments

{maidr-1.2.0.dist-info → maidr-1.2.2.dist-info}/RECORD

@@ -1,4 +1,4 @@
-maidr/__init__.py,sha256=LU9S8UQwrqi4lCD1ksf-ye6PN8AeNXzjiXBEwwCp5iI,399
+maidr/__init__.py,sha256=8RJF5FjMzhNWOpVQkU_BCfkLKaWvTrrQeTIat9_SXJE,399
 maidr/api.py,sha256=F43mXWsxc7tHdlZqbRlEWkc-RjJVo_zgxCn3NiLBY58,1764
 maidr/core/__init__.py,sha256=WgxLpSEYMc4k3OyEOf1shOxfEq0ASzppEIZYmE91ThQ,25
 maidr/core/context_manager.py,sha256=6cT7ZGOApSpC-SLD2XZWWU_H08i-nfv-JUlzXOtvWYw,3374
@@ -16,7 +16,7 @@ maidr/core/plot/candlestick.py,sha256=8YKjNmxJx7TEvaH5v4lYDzA11VHpVHpZrv9EJG2Ter
 maidr/core/plot/grouped_barplot.py,sha256=bRcQcvwkF3Q3aZ3PlhbZ6bHI_AfcqdKUMVvlLL94wXM,2078
 maidr/core/plot/heatmap.py,sha256=yMS-31tS2GW4peds9LtZesMxmmTV_YfqYO5M_t5KasQ,2521
 maidr/core/plot/histogram.py,sha256=QV5W-6ZJQQcZsrM91JJBX-ONktJzH7yg_et5_bBPfQQ,1525
-maidr/core/plot/lineplot.py,sha256=_Sg290rk5h1LvtdaY_VKraTLXyiXGi58t-WxLFht5Ds,3331
+maidr/core/plot/lineplot.py,sha256=C3xz6uWXYM_mbTq_geb5bP0JdvhcQf6cpfTs78Y6fCM,3852
 maidr/core/plot/maidr_plot.py,sha256=B6hjsu-jSWlevEqJawgwjMOJr51nBBNh7yqJdSTkNhw,3681
 maidr/core/plot/maidr_plot_factory.py,sha256=5SC8Nc3IfVYn-jQU_SD9vw7R5yhTHTBgQwGnAc7_DmA,2507
 maidr/core/plot/regplot.py,sha256=b7u6bGTz1IxKahplNUrfwIr_OGSwMJ2BuLgFAVjL0s0,2744
@@ -33,7 +33,7 @@ maidr/patch/heatmap.py,sha256=uxLLg530Ql9KVC5rxk8vnwPHXBWWHwYgJRkyHY-tJzs,1048
 maidr/patch/highlight.py,sha256=I1dGFHJAnVd0AHVnMJzk_TE8BC8Uv-I6fTzSrJLU5QM,1155
 maidr/patch/histogram.py,sha256=k3N0RUf1SQ2402pwbaY5QyS98KnLWvr9glCHQw9NTko,2378
 maidr/patch/kdeplot.py,sha256=qv-OKzuop2aTrkZgUe2OnLxvV-KMyeXt1Td0_uZeHzE,2338
-maidr/patch/lineplot.py,sha256=BCc3tyTrQbdLPUDWIO7hu4h1W71JRPSt2d4tYP54Li4,1085
+maidr/patch/lineplot.py,sha256=og42V0tWBKCnf6idT3pLsIj3QBvKjg8aUN-k1udPRVw,1901
 maidr/patch/regplot.py,sha256=Ciz43C5XZfWK6wtVWrlV0WNz4R__rcgdqVE9OCaXXRk,3236
 maidr/patch/scatterplot.py,sha256=kln6zZwjVsdQzICalo-RnBOJrx1BnIB2xYUwItHvSNY,525
 maidr/util/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -46,7 +46,7 @@ maidr/util/regression_line_utils.py,sha256=P8RQLixTby2JLz73XZgNiu96C2Ct3pNe4ENRW
 maidr/util/svg_utils.py,sha256=2gyzBtNKFHs0utrw1iOlxTmznzivOWQMV2aW8zu2c8E,1442
 maidr/widget/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 maidr/widget/shiny.py,sha256=wrrw2KAIpE_A6CNQGBtNHauR1DjenA_n47qlFXX9_rk,745
-maidr-1.2.0.dist-info/LICENSE,sha256=IwGE9guuL-ryRPEKi6wFPI_zOhg7zDZbTYuHbSt_SAk,35823
-maidr-1.2.0.dist-info/METADATA,sha256=ktazg6njCh_Y6Eb6fkNd4vbR2gPNHli-WtAC124v3s4,2796
-maidr-1.2.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-maidr-1.2.0.dist-info/RECORD,,
+maidr-1.2.2.dist-info/LICENSE,sha256=IwGE9guuL-ryRPEKi6wFPI_zOhg7zDZbTYuHbSt_SAk,35823
+maidr-1.2.2.dist-info/METADATA,sha256=Nd6NYe790oDdPYo_iu51Uk18cOOSKZFw3845AI_MKec,2796
+maidr-1.2.2.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+maidr-1.2.2.dist-info/RECORD,,