CI: test upstream dev (#133)

* CI: test upstream dev

* remove timeout

* avoid unwritable array

* use variable

* changelog

Author: mathause

.github/workflows/upstream-dev-ci.yaml

@@ -0,0 +1,78 @@
+name: CI Upstream
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+    types: [opened, reopened, synchronize, labeled]
+  schedule:
+    - cron: "0 0 * * *" # Daily “At 00:00” UTC
+    # TODO: switch to weekly once I know this works
+    # - cron: "0 17 * * 1" # “At 17:00 on Monday” UTC
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+
+  upstream-dev:
+    name: upstream-dev
+    runs-on: ubuntu-latest
+    if: |
+        always()
+        && (
+            github.event_name == 'schedule'
+            || contains( github.event.pull_request.labels.*.name, 'run-upstream')
+        )
+    defaults:
+      run:
+        shell: bash -l {0}
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ["3.12"]
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0 # Fetch all history for all branches and tags.
+      - name: Set up conda environment
+        uses: mamba-org/setup-micromamba@v2
+        with:
+          environment-file: ci/requirements/environment.yml
+          environment-name: mplotutils-tests
+          create-args: >-
+            python=${{ matrix.python-version }}
+            pytest-reportlog
+            conda
+      - name: Install upstream versions
+        run: |
+          bash ci/install-upstream-wheels.sh
+      - name: Install mplotutils
+        run: |
+          python -m pip install --no-deps -e .
+      - name: Version info
+        run: |
+          conda info -a
+          conda list
+
+      - name: Import mplotutils
+        run: |
+          python -c 'import mplotutils'
+      - name: Run Tests
+        if: success()
+        id: status
+        run: |
+          python -m pytest -rf \
+            --report-log output-${{ matrix.python-version }}-log.jsonl
+      - name: Generate and publish the report
+        if: |
+          failure()
+          && steps.status.outcome == 'failure'
+          && github.event_name == 'schedule'
+          && github.repository_owner == 'regionmask'
+        uses: xarray-contrib/issue-from-pytest-log@v1
+        with:
+          log-path: output-${{ matrix.python-version }}-log.jsonl

CHANGELOG.md

@@ -17,7 +17,7 @@
 
 ### Enhancements
 
-- Added convinience functions to draw hatches and add stippling:
+- Added convenience functions to draw hatches and add stippling:
 
   1. `mpu.hatch`: for regular axes
   2. `mpu.hatch_map`: for cartopy GeoAxes
@@ -28,6 +28,7 @@
 - Enable passing `AxesGrid` (from `mpl_toolkits.axes_grid1`) to `set_map_layout` ([#116](https://github.com/mathause/mplotutils/pull/116)).
 - Raise more informative error when a wrong type is passed to `set_map_layout` ([#121](https://github.com/mathause/mplotutils/pull/121)).
 - `set_map_layout` now raises an explicit error when the figure contains SubFigure ([#121](https://github.com/mathause/mplotutils/pull/121)).
+- Test upstream dependencies and fix compatibility with the upcoming pandas v3  ([#133](https://github.com/mathause/mplotutils/pull/133)).
 
 ## v0.5.0 (27.03.2024)
 

ci/install-upstream-wheels.sh

@@ -0,0 +1,27 @@
+#!/usr/bin/env bash
+
+# forcibly remove packages to avoid artifacts
+conda uninstall -y --force \
+  cartopy \
+  matplotlib-base \
+  numpy \
+  packaging \
+  pandas \
+  xarray
+# to limit the runtime of Upstream CI
+python -m pip install \
+    -i https://pypi.anaconda.org/scientific-python-nightly-wheels/simple \
+    --no-deps \
+    --pre \
+    --upgrade \
+    matplotlib \
+    numpy \
+    pandas \
+    shapely \
+    xarray
+python -m pip install \
+    --no-deps \
+    --upgrade \
+    git+https://github.com/mwaskom/seaborn \
+    git+https://github.com/pypa/packaging \
+    git+https://github.com/SciTools/cartopy

mplotutils/cartopy_utils.py

@@ -118,14 +118,18 @@ def cyclic_dataarray(obj, coord="lon"):
     obj = obj.pad({coord: (0, 1)}, mode="wrap")
 
     # extrapolate the coords
-    lon = obj[coord]
-
-    diff = lon.isel({coord: slice(None, -1)}).diff(coord)
+    diff = obj[coord].isel({coord: slice(None, -1)}).diff(coord)
 
     if not np.allclose(diff, diff[0]):
         raise ValueError(f"The coordinate '{coord}' must be equally spaced")
 
-    lon.data[-1] = lon[-2] + diff[0]
+    # the data is not writable (pandas 3) - a copy is required
+    lon = obj[coord].variable
+
+    arr = np.array(lon.data)
+    arr[-1] = arr[-2] + diff[0]
+
+    lon = type(lon)(lon.dims, arr, attrs=lon.attrs, encoding=lon.encoding)
 
     return obj.assign_coords({coord: lon})
 

mplotutils/tests/test_cyclic_dataarray.py

@@ -28,13 +28,15 @@ def test_cyclic_dataarray_not_equally_spaced(as_dataset):
 @pytest.mark.parametrize("as_dataset", (True, False))
 def test_cyclic_dataarray(as_dataset):
     data = [[1, 2, 3], [4, 5, 6]]
-    da = xr.DataArray(
-        data, dims=("y", "x"), coords={"y": [1, 2], "x": [0, 1, 2]}, name="data"
-    )
+
+    y = xr.Variable("y", [1, 2], attrs={"foo": "bar"})
+    x = xr.Variable("x", [0, 1, 2], attrs={"foo": "bar"})
+    da = xr.DataArray(data, dims=("y", "x"), coords={"y": y, "x": x}, name="data")
 
     expected = [[1, 2, 3, 1], [4, 5, 6, 4]]
+    x = xr.Variable("x", [0, 1, 2, 3], attrs={"foo": "bar"})
     da_expected = xr.DataArray(
-        expected, dims=("y", "x"), coords={"y": [1, 2], "x": [0, 1, 2, 3]}, name="data"
+        expected, dims=("y", "x"), coords={"y": y, "x": x}, name="data"
     )
 
     data = da.to_dataset() if as_dataset else da