plopp 25.4.1__tar.gz → 25.5.0__tar.gz

{plopp-25.4.1 → plopp-25.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: plopp
-Version: 25.4.1
+Version: 25.5.0
 Summary: Visualization library for Scipp
 Author: Scipp contributors
 License: BSD 3-Clause License

{plopp-25.4.1 → plopp-25.5.0}/requirements/base.txt

@@ -32,7 +32,7 @@ pyparsing==3.2.1
     # via matplotlib
 python-dateutil==2.9.0.post0
     # via matplotlib
-scipp==25.4.0
+scipp==25.5.0
     # via -r base.in
 six==1.17.0
     # via python-dateutil

{plopp-25.4.1 → plopp-25.5.0}/requirements/basetest.txt

@@ -134,7 +134,7 @@ pytz==2025.1
     # via pandas
 requests==2.32.3
     # via pooch
-scipp==25.4.0
+scipp==25.5.0
     # via -r basetest.in
 scipy==1.15.2
     # via -r basetest.in

{plopp-25.4.1 → plopp-25.5.0}/src/plopp/graphics/graphicalview.py

@@ -21,6 +21,15 @@ def _none_if_not_finite(x: float | None) -> float | int | None:
     return x if np.isfinite(x) else None
 
 
+def _make_range(
+    old: tuple[float, float], new: tuple[float, float]
+) -> tuple[float | None, float | None]:
+    new = (_none_if_not_finite(new[0]), _none_if_not_finite(new[1]))
+    if (old is not None) and (None not in old) and (old[0] > old[1]):
+        new = (new[1], new[0])
+    return new
+
+
 class GraphicalView(View):
     """
     Base class for graphical 1d and 2d views.
@@ -113,18 +122,15 @@ class GraphicalView(View):
             bbox = bbox.union(artist.bbox(**scales))
         self.bbox = bbox
         self.bbox = self.bbox.override(self.canvas.bbox)
-        self.canvas.xrange = (
-            _none_if_not_finite(self.bbox.xmin),
-            _none_if_not_finite(self.bbox.xmax),
+        self.canvas.xrange = _make_range(
+            old=self.canvas.xrange, new=(self.bbox.xmin, self.bbox.xmax)
         )
-        self.canvas.yrange = (
-            _none_if_not_finite(self.bbox.ymin),
-            _none_if_not_finite(self.bbox.ymax),
+        self.canvas.yrange = _make_range(
+            old=self.canvas.yrange, new=(self.bbox.ymin, self.bbox.ymax)
         )
         if hasattr(self.canvas, 'zrange'):
-            self.canvas.zrange = (
-                _none_if_not_finite(self.bbox.zmin),
-                _none_if_not_finite(self.bbox.zmax),
+            self.canvas.zrange = _make_range(
+                old=self.canvas.zrange, new=(self.bbox.zmin, self.bbox.zmax)
             )
 
     def update(self, *args, **kwargs) -> None:

{plopp-25.4.1 → plopp-25.5.0}/src/plopp/plotting/common.py

@@ -2,7 +2,7 @@
 # Copyright (c) 2023 Scipp contributors (https://github.com/scipp)
 
 import warnings
-from collections.abc import Callable
+from collections.abc import Callable, Iterable
 from typing import Any
 
 import numpy as np
@@ -76,8 +76,30 @@ def to_variable(obj) -> sc.Variable:
     return out
 
 
+def _ensure_data_array_coords(
+    da: sc.DataArray, coords: list[str] | None
+) -> sc.DataArray:
+    if coords is None:
+        coords = list(da.dims)
+    elif missing := set(coords) - set(da.coords.keys()):
+        raise ValueError(f"Specified coords do not exist: {missing}")
+
+    # Remove unused coords
+    da = da.drop_coords(list(set(da.coords) - set(coords)))
+    # Assign dim coords where missing. The above checks ensure that all missing
+    # coords are dim coords, i.e., that `name in out.dims`.
+    da = da.assign_coords(
+        {
+            name: sc.arange(name, da.sizes[name], unit=None)
+            for name in set(coords) - set(da.coords)
+        }
+    )
+    return da
+
+
 def to_data_array(
     obj: Plottable | list,
+    coords: list[str] | None = None,
 ) -> sc.DataArray:
     """
     Convert an input to a DataArray, potentially adding fake coordinates if they are
@@ -87,6 +109,8 @@ def to_data_array(
     ----------
     obj:
         The input object to be converted.
+    coords:
+        If supplied, use these coords instead of the input's dimension coordinates.
     """
     out = _maybe_to_variable(obj)
     if isinstance(out, sc.Variable):
@@ -94,15 +118,13 @@ def to_data_array(
     out = from_compatible_lib(out)
     if not isinstance(out, sc.DataArray):
         raise TypeError(f"Cannot convert input of type {type(obj)} to a DataArray.")
-    out = out.copy(deep=False)
-    for dim, size in out.sizes.items():
-        if dim not in out.coords:
-            out.coords[dim] = sc.arange(dim, size, unit=None)
-        if not out.coords[dim].dims:
+    out = _ensure_data_array_coords(out, coords)
+    for name, coord in out.coords.items():
+        if not coord.dims:
             raise ValueError(
                 "Input data cannot be plotted: it has a scalar coordinate along "
-                f"dimension {dim}. Consider dropping this coordinate before plotting. "
-                f"Use ``data.drop_coords('{dim}').plot()``."
+                f"dimension {name}. Consider dropping this coordinate before plotting. "
+                f"Use ``data.drop_coords('{name}').plot()``."
             )
     for name in out.coords:
         other_dims = [dim for dim in out.coords[name].dims if dim not in out.dims]
@@ -171,7 +193,7 @@ def preprocess(
     obj: Plottable | list,
     name: str | None = None,
     ignore_size: bool = False,
-    coords: list[str] | None = None,
+    coords: Iterable[str] | str | None = None,
 ) -> sc.DataArray:
     """
     Pre-process input data for plotting.
@@ -192,7 +214,12 @@ def preprocess(
     coords:
         If supplied, use these coords instead of the input's dimension coordinates.
     """
-    out = to_data_array(obj)
+    if isinstance(coords, str):
+        coords = [coords]
+    elif coords is not None:
+        coords = list(coords)
+
+    out = to_data_array(obj, coords)
     check_not_binned(out)
     check_allowed_dtypes(out)
     if name is not None:
@@ -201,8 +228,6 @@ def preprocess(
         _check_size(out)
     if coords is not None:
         renamed_dims = {}
-        if isinstance(coords, str):
-            coords = [coords]
         for dim in coords:
             underlying = out.coords[dim].dims[-1]
             if underlying in renamed_dims:

{plopp-25.4.1 → plopp-25.5.0}/src/plopp.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: plopp
-Version: 25.4.1
+Version: 25.5.0
 Summary: Visualization library for Scipp
 Author: Scipp contributors
 License: BSD 3-Clause License

{plopp-25.4.1 → plopp-25.5.0}/tests/graphics/canvas_test.py

@@ -148,6 +148,22 @@ class TestCanvasNo3d:
         assert canvas.xscale == 'linear'
         assert canvas.yscale == 'log'
 
+    def test_xrange_order_preserved(self, set_backend, backend, figure, data):
+        da = data()
+        fig = figure(Node(da))
+        fig.canvas.xrange = (52.0, -5.0)
+        fig.view.autoscale()
+        new_range = fig.canvas.xrange
+        assert new_range[0] > new_range[1]
+
+    def test_yrange_order_preserved(self, set_backend, backend, figure, data):
+        da = data()
+        fig = figure(Node(da))
+        fig.canvas.yrange = (61.0, -6.6)
+        fig.view.autoscale()
+        new_range = fig.canvas.yrange
+        assert new_range[0] > new_range[1]
+
 
 CASES1DINTERACTIVE = {
     k: c for k, c in CASES.items() if (c[1] is linefigure and c[0][1] != 'mpl-static')

{plopp-25.4.1 → plopp-25.5.0}/tests/plotting/common_test.py

@@ -25,6 +25,14 @@ def test_preprocess_use_non_dimension_coords():
     assert out.coords['yy2'].max() == 3.3 * da.coords['yy'].max()
 
 
+def test_preprocess_use_non_dimension_coord_single_str():
+    da = data_array(ndim=1)
+    da.coords['xx2'] = 7.5 * da.coords['xx']
+    out = preprocess(da, coords='xx2')
+    assert set(out.dims) == {'xx2'}
+    assert out.coords['xx2'].max() == 7.5 * da.coords['xx'].max()
+
+
 def test_preprocess_warns_when_coordinate_is_not_sorted():
     da = data_array(ndim=1)
     unsorted = sc.concat([da['xx', 20:], da['xx', :20]], dim='xx')
@@ -40,12 +48,12 @@ def test_preprocess_no_warning_if_dtype_cannot_be_sorted():
         dims=['xx'], values=np.random.random((da.sizes['xx'], 3))
     )
     out = preprocess(da)  # no warning should be emitted
-    assert 'vecs' in out.coords
+    assert 'vecs' not in out.coords  # coord doesn't participate in plot
     da.coords['strings'] = sc.array(
         dims=['xx'], values=list('ba' * (da.sizes['xx'] // 2))
     )
     out = preprocess(da)  # no warning should be emitted
-    assert 'strings' in out.coords
+    assert 'strings' not in out.coords  # coord doesn't participate in plot
 
 
 @pytest.mark.parametrize(
@@ -70,3 +78,26 @@ def test_preprocess_raises_for_unsupported_dtype(dtype_and_shape):
         match=f'The input has dtype {dtype} which is not supported by Plopp',
     ):
         preprocess(v)
+
+
+def test_preprocess_drops_coords_that_are_not_plotted_dim_coords():
+    da = data_array(ndim=2)
+    da.coords['xx2'] = 7.5 * da.coords['xx']
+    da.coords['yy2'] = 3.3 * da.coords['yy']
+    out = preprocess(da)
+    assert set(out.coords) == {'xx', 'yy'}
+
+
+def test_preprocess_drops_coords_that_are_not_plotted_custom_coords():
+    da = data_array(ndim=2)
+    da.coords['xx2'] = 7.5 * da.coords['xx']
+    da.coords['yy2'] = 3.3 * da.coords['yy']
+    out = preprocess(da, coords=['yy2', 'xx'])
+    assert set(out.coords) == {'xx', 'yy2'}
+
+
+def test_preprocess_drops_coords_that_are_not_plotted_custom_coord_single_str():
+    da = data_array(ndim=1)
+    da.coords['xx2'] = 7.5 * da.coords['xx']
+    out = preprocess(da, coords='xx')
+    assert set(out.coords) == {'xx'}

{plopp-25.4.1 → plopp-25.5.0}/tests/plotting/plot_1d_test.py

@@ -414,6 +414,16 @@ def test_plot_1d_datetime_length_1():
     pp.plot(da)
 
 
+def test_plot_1d_extra_datetime_coord_binedges():
+    da = data_array(ndim=1)
+    dim = da.dim
+    size = da.sizes[dim]
+    da.coords['times'] = (sc.arange('t', 2 * size, unit='s') + sc.epoch(unit='s')).fold(
+        dim='t', sizes={dim: size, 't': 2}
+    )
+    pp.plot(da)
+
+
 def test_plot_1d_data_with_errorbars():
     da = data_array(ndim=1, variances=True)
     p = da.plot()