xarray-plotly 0.0.11__tar.gz → 0.0.13__tar.gz

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/.github/workflows/ci.yml

@@ -31,6 +31,9 @@ jobs:
       - name: Format check
         run: uv run ruff format --check .
 
+      - name: Type check
+        run: uv run mypy xarray_plotly
+
       - name: Test
         run: uv run pytest --cov=xarray_plotly --cov-report=xml
 

xarray_plotly-0.0.13/.github/workflows/pr-title.yml

@@ -0,0 +1,33 @@
+name: PR Title Check
+
+on:
+  pull_request:
+    types: [opened, edited, synchronize, reopened]
+
+jobs:
+  validate:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: amannn/action-semantic-pull-request@v5
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          # Require conventional commit format: type(scope): description
+          # Types allowed (default set)
+          types: |
+            feat
+            fix
+            docs
+            style
+            refactor
+            perf
+            test
+            build
+            ci
+            chore
+            revert
+          # Scope is optional
+          requireScope: false
+          # Subject (description) requirements
+          subjectPattern: ^.+$
+          subjectPatternError: "PR title must have a description after the type"

xarray_plotly-0.0.13/.github/workflows/release-please.yml

@@ -0,0 +1,19 @@
+name: Release Please
+
+on:
+  push:
+    branches:
+      - main
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  release-please:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: googleapis/release-please-action@v4
+        with:
+          config-file: .release-please-config.json
+          manifest-file: .release-please-manifest.json

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/.github/workflows/release.yml

@@ -10,7 +10,6 @@ jobs:
     runs-on: ubuntu-latest
     permissions:
       id-token: write  # for trusted publishing
-      contents: write  # for creating GitHub release
 
     steps:
       - uses: actions/checkout@v6
@@ -23,9 +22,3 @@ jobs:
 
       - name: Publish to PyPI
         uses: pypa/gh-action-pypi-publish@release/v1
-
-      - name: Create GitHub Release
-        uses: softprops/action-gh-release@v2
-        with:
-          generate_release_notes: true
-          files: dist/*

xarray_plotly-0.0.13/.release-please-config.json

@@ -0,0 +1,13 @@
+{
+  "$schema": "https://raw.githubusercontent.com/googleapis/release-please/main/schemas/config.json",
+  "bootstrap-sha": "3c1b9d991a58c2369fe4817ad2f3d555aaab969e",
+  "include-component-in-tag": false,
+  "packages": {
+    ".": {
+      "release-type": "simple",
+      "package-name": "xarray_plotly",
+      "bump-patch-for-minor-pre-major": true,
+      "changelog-path": "CHANGELOG.md"
+    }
+  }
+}

xarray_plotly-0.0.13/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.0.13"
+}

xarray_plotly-0.0.13/CHANGELOG.md

@@ -0,0 +1,8 @@
+# Changelog
+
+## [0.0.13](https://github.com/FBumann/xarray_plotly/compare/v0.0.12...v0.0.13) (2026-01-28)
+
+
+### Bug Fixes
+
+* secondary y axis match and visibility when faceted ([#32](https://github.com/FBumann/xarray_plotly/issues/32)) ([10da229](https://github.com/FBumann/xarray_plotly/commit/10da229282377a229054321e8ff98e3fd6825939))

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xarray_plotly
-Version: 0.0.11
+Version: 0.0.13
 Summary: Interactive Plotly Express plotting accessor for xarray
 Author: Felix
 License: MIT
@@ -84,6 +84,25 @@ fig = xpx(da).line()
 
 Full documentation: [https://fbumann.github.io/xarray_plotly](https://fbumann.github.io/xarray_plotly)
 
+## Roadmap
+
+Planned additions (contributions welcome):
+
+**New plot types**
+- `histogram()` — distribution of DataArray values
+- `violin()` — richer distribution visualization than box plots
+- `density_heatmap()` — 2D histograms (x and y as dimensions)
+- `density_contour()` — 2D density contours (x and y as dimensions)
+
+**Enhancements**
+- WebGL rendering option for `line()` and `scatter()` (large datasets)
+
+**Figure utilities** (facet/animation-aware)
+- `fill_between()` — fill area between two traces (uncertainty bands)
+- `sync_axes()` — consistent axis ranges across facets and animation frames
+- `add_secondary_x()` — secondary x-axis (like `add_secondary_y()`)
+- `stack()` — vertically stack separate figures into subplots
+
 ## License
 
 MIT

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/README.md

@@ -42,6 +42,25 @@ fig = xpx(da).line()
 
 Full documentation: [https://fbumann.github.io/xarray_plotly](https://fbumann.github.io/xarray_plotly)
 
+## Roadmap
+
+Planned additions (contributions welcome):
+
+**New plot types**
+- `histogram()` — distribution of DataArray values
+- `violin()` — richer distribution visualization than box plots
+- `density_heatmap()` — 2D histograms (x and y as dimensions)
+- `density_contour()` — 2D density contours (x and y as dimensions)
+
+**Enhancements**
+- WebGL rendering option for `line()` and `scatter()` (large datasets)
+
+**Figure utilities** (facet/animation-aware)
+- `fill_between()` — fill area between two traces (uncertainty bands)
+- `sync_axes()` — consistent axis ranges across facets and animation frames
+- `add_secondary_x()` — secondary x-axis (like `add_secondary_y()`)
+- `stack()` — vertically stack separate figures into subplots
+
 ## License
 
 MIT

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/docs/examples/kwargs.ipynb

@@ -159,6 +159,71 @@
     "xpx(change).imshow(color_continuous_scale=\"RdBu_r\", color_continuous_midpoint=0)"
    ]
   },
+  {
+   "cell_type": "markdown",
+   "metadata": {},
+   "source": [
+    "## colors (unified parameter)\n",
+    "\n",
+    "The `colors` parameter provides a simpler way to set colors without remembering the exact Plotly parameter name. It automatically maps to the correct parameter based on the input type:\n",
+    "\n",
+    "| Input | Maps To |\n",
+    "|-------|---------|\n",
+    "| `\"Viridis\"` (continuous scale name) | `color_continuous_scale` |\n",
+    "| `\"D3\"` (qualitative palette name) | `color_discrete_sequence` |\n",
+    "| `[\"red\", \"blue\"]` (list) | `color_discrete_sequence` |\n",
+    "| `{\"A\": \"red\"}` (dict) | `color_discrete_map` |"
+   ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "# Named qualitative palette\n",
+    "xpx(stocks).line(colors=\"D3\")"
+   ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "# List of custom colors\n",
+    "xpx(stocks).line(colors=[\"#E63946\", \"#457B9D\", \"#2A9D8F\", \"#E9C46A\", \"#F4A261\"])"
+   ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "# Dict for explicit mapping\n",
+    "xpx(stocks).line(\n",
+    "    colors={\n",
+    "        \"GOOG\": \"red\",\n",
+    "        \"AAPL\": \"blue\",\n",
+    "        \"AMZN\": \"green\",\n",
+    "        \"FB\": \"purple\",\n",
+    "        \"NFLX\": \"orange\",\n",
+    "        \"MSFT\": \"brown\",\n",
+    "    }\n",
+    ")"
+   ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "# Continuous scale for heatmaps\n",
+    "xpx(stocks).imshow(colors=\"Plasma\")"
+   ]
+  },
   {
    "cell_type": "markdown",
    "metadata": {},

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/tests/test_accessor.py

@@ -401,3 +401,128 @@ class TestImshowBounds:
         coloraxis = fig.layout.coloraxis
         assert coloraxis.cmin == 0.0
         assert coloraxis.cmax == 70.0
+
+
+class TestColorsParameter:
+    """Tests for the unified colors parameter."""
+
+    @pytest.fixture(autouse=True)
+    def setup(self) -> None:
+        """Create test DataArrays."""
+        self.da = xr.DataArray(
+            np.random.rand(10, 3),
+            dims=["time", "city"],
+            coords={"city": ["A", "B", "C"]},
+        )
+
+    def test_colors_list_sets_discrete_sequence(self) -> None:
+        """Test that a list of colors sets color_discrete_sequence."""
+        fig = self.da.plotly.line(colors=["red", "blue", "green"])
+        # Check that traces have the expected colors
+        assert len(fig.data) == 3
+        assert fig.data[0].line.color == "red"
+        assert fig.data[1].line.color == "blue"
+        assert fig.data[2].line.color == "green"
+
+    def test_colors_dict_sets_discrete_map(self) -> None:
+        """Test that a dict sets color_discrete_map."""
+        fig = self.da.plotly.line(colors={"A": "red", "B": "blue", "C": "green"})
+        # Traces should be colored according to the mapping
+        assert len(fig.data) == 3
+        # Find traces by name and check their color
+        colors_by_name = {trace.name: trace.line.color for trace in fig.data}
+        assert colors_by_name["A"] == "red"
+        assert colors_by_name["B"] == "blue"
+        assert colors_by_name["C"] == "green"
+
+    def test_colors_continuous_scale_string(self) -> None:
+        """Test that a continuous scale name sets color_continuous_scale."""
+        da = xr.DataArray(
+            np.random.rand(50, 2),
+            dims=["point", "coord"],
+            coords={"coord": ["x", "y"]},
+        )
+        fig = da.plotly.scatter(y="coord", x="point", color="value", colors="Viridis")
+        # Plotly Express uses coloraxis in the layout for continuous scales
+        # Check that the colorscale was applied to the coloraxis
+        assert fig.layout.coloraxis.colorscale is not None
+        colorscale = fig.layout.coloraxis.colorscale
+        # Viridis should be in the colorscale definition
+        assert any("viridis" in str(c).lower() for c in colorscale) or len(colorscale) > 0
+
+    def test_colors_qualitative_palette_string(self) -> None:
+        """Test that a qualitative palette name sets color_discrete_sequence."""
+        import plotly.express as px
+
+        fig = self.da.plotly.line(colors="D3")
+        # D3 palette should be applied - check first trace color is from D3
+        d3_colors = px.colors.qualitative.D3
+        assert fig.data[0].line.color in d3_colors
+
+    def test_colors_ignored_with_warning_when_px_kwargs_present(self) -> None:
+        """Test that colors is ignored with warning when color_* kwargs are present."""
+        import warnings
+
+        with warnings.catch_warnings(record=True) as w:
+            warnings.simplefilter("always")
+            fig = self.da.plotly.line(
+                colors="D3", color_discrete_sequence=["orange", "purple", "cyan"]
+            )
+            # Should have raised a warning about colors being ignored
+            assert any(
+                "colors" in str(m.message).lower() and "ignored" in str(m.message).lower()
+                for m in w
+            ), "Expected warning about 'colors' being 'ignored' not found"
+            # The explicit px_kwargs should take precedence
+            assert fig.data[0].line.color == "orange"
+
+    def test_colors_none_uses_defaults(self) -> None:
+        """Test that colors=None uses Plotly defaults."""
+        fig1 = self.da.plotly.line(colors=None)
+        fig2 = self.da.plotly.line()
+        # Both should produce the same result
+        assert fig1.data[0].line.color == fig2.data[0].line.color
+
+    def test_colors_works_with_bar(self) -> None:
+        """Test colors parameter with bar chart."""
+        fig = self.da.plotly.bar(colors=["#e41a1c", "#377eb8", "#4daf4a"])
+        assert fig.data[0].marker.color == "#e41a1c"
+
+    def test_colors_works_with_area(self) -> None:
+        """Test colors parameter with area chart."""
+        fig = self.da.plotly.area(colors=["red", "green", "blue"])
+        assert len(fig.data) == 3
+
+    def test_colors_works_with_scatter(self) -> None:
+        """Test colors parameter with scatter plot."""
+        fig = self.da.plotly.scatter(colors=["red", "green", "blue"])
+        assert len(fig.data) == 3
+
+    def test_colors_works_with_imshow(self) -> None:
+        """Test colors parameter with imshow (continuous scale)."""
+        da = xr.DataArray(np.random.rand(10, 10), dims=["y", "x"])
+        fig = da.plotly.imshow(colors="RdBu")
+        # Plotly Express uses coloraxis in the layout for continuous scales
+        assert fig.layout.coloraxis.colorscale is not None
+        colorscale = fig.layout.coloraxis.colorscale
+        # RdBu should be in the colorscale definition
+        assert any("rdbu" in str(c).lower() for c in colorscale) or len(colorscale) > 0
+
+    def test_colors_works_with_pie(self) -> None:
+        """Test colors parameter with pie chart."""
+        da = xr.DataArray([30, 40, 30], dims=["category"], coords={"category": ["A", "B", "C"]})
+        fig = da.plotly.pie(colors={"A": "red", "B": "blue", "C": "green"})
+        assert isinstance(fig, go.Figure)
+
+    def test_colors_works_with_dataset(self) -> None:
+        """Test colors parameter works with Dataset accessor."""
+        ds = xr.Dataset(
+            {
+                "temp": (["time"], np.random.rand(10)),
+                "precip": (["time"], np.random.rand(10)),
+            }
+        )
+        fig = ds.plotly.line(colors=["red", "blue"])
+        assert len(fig.data) == 2
+        assert fig.data[0].line.color == "red"
+        assert fig.data[1].line.color == "blue"

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/tests/test_figures.py

@@ -505,8 +505,8 @@ class TestAddSecondaryYFacets:
 
         combined = add_secondary_y(base, secondary, secondary_y_title="Custom Title")
 
-        # Title should be on the first secondary axis
-        assert combined.layout.yaxis4.title.text == "Custom Title"
+        # Title should be on the rightmost secondary axis (yaxis6 for 3 facets)
+        assert combined.layout.yaxis6.title.text == "Custom Title"
 
 
 class TestAddSecondaryYAnimation:

{xarray_plotly-0.0.11 → xarray_plotly-0.0.13}/xarray_plotly/__init__.py

@@ -52,7 +52,7 @@ from xarray import DataArray, Dataset, register_dataarray_accessor, register_dat
 
 from xarray_plotly import config
 from xarray_plotly.accessor import DataArrayPlotlyAccessor, DatasetPlotlyAccessor
-from xarray_plotly.common import SLOT_ORDERS, auto
+from xarray_plotly.common import SLOT_ORDERS, Colors, auto
 from xarray_plotly.figures import (
     add_secondary_y,
     overlay,
@@ -61,6 +61,7 @@ from xarray_plotly.figures import (
 
 __all__ = [
     "SLOT_ORDERS",
+    "Colors",
     "add_secondary_y",
     "auto",
     "config",
@@ -110,5 +111,5 @@ def xpx(data: DataArray | Dataset) -> DataArrayPlotlyAccessor | DatasetPlotlyAcc
 __version__ = version("xarray_plotly")
 
 # Register the accessors
-register_dataarray_accessor("plotly")(DataArrayPlotlyAccessor)
-register_dataset_accessor("plotly")(DatasetPlotlyAccessor)
+register_dataarray_accessor("plotly")(DataArrayPlotlyAccessor)  # type: ignore[no-untyped-call]
+register_dataset_accessor("plotly")(DatasetPlotlyAccessor)  # type: ignore[no-untyped-call]