Added daskgeopandas check to is_geodataframe function

envs/py3.10-tests.yaml

@@ -66,6 +66,7 @@ dependencies:
   - scipy>=1.5.3
   - selenium>=3.141.0
   - setuptools_scm>=6
+  - spatialpandas
   - spatialpandas>=0.4.3
   - sqlglot
   - streamz>=0.3.0

envs/py3.11-tests.yaml

@@ -66,6 +66,7 @@ dependencies:
   - scipy>=1.5.3
   - selenium>=3.141.0
   - setuptools_scm>=6
+  - spatialpandas
   - spatialpandas>=0.4.3
   - sqlglot
   - streamz>=0.3.0

envs/py3.12-tests.yaml

@@ -66,6 +66,7 @@ dependencies:
   - scipy>=1.5.3
   - selenium>=3.141.0
   - setuptools_scm>=6
+  - spatialpandas
   - spatialpandas>=0.4.3
   - sqlglot
   - streamz>=0.3.0

envs/py3.9-tests.yaml

@@ -65,6 +65,7 @@ dependencies:
   - scipy>=1.5.3
   - selenium>=3.141.0
   - setuptools_scm>=6
+  - spatialpandas
   - spatialpandas>=0.4.3
   - sqlglot
   - streamz>=0.3.0

hvplot/tests/testutil.py

@@ -7,6 +7,13 @@
 import panel as pn
 import pytest
 
+try:
+    import dask.dataframe as dd
+    import spatialpandas as spd
+except ImportError:
+    spd = None
+    dd = None
+
 from unittest import TestCase, SkipTest
 
 from hvplot.util import (
@@ -16,6 +23,7 @@
     process_xarray,
     _convert_col_names_to_str,
     instantiate_crs_str,
+    is_geodataframe,
 )
 
 
@@ -184,7 +192,7 @@ def test_process_xarray_dataset_with_x_as_derived_datetime(self):
 
 class TestDynamicArgs(TestCase):
     def test_dynamic_and_static(self):
-        from ..util import process_dynamic_args
+        from hvplot.util import process_dynamic_args
 
         x = 'sepal_width'
         y = pn.widgets.Select(
@@ -198,7 +206,7 @@ def test_dynamic_and_static(self):
         assert arg_deps == []
 
     def test_dynamic_kwds(self):
-        from ..util import process_dynamic_args
+        from hvplot.util import process_dynamic_args
 
         x = 'sepal_length'
         y = 'sepal_width'
@@ -211,7 +219,7 @@ def test_dynamic_kwds(self):
         assert arg_deps == []
 
     def test_fn_kwds(self):
-        from ..util import process_dynamic_args
+        from hvplot.util import process_dynamic_args
 
         x = 'sepal_length'
         y = 'sepal_width'
@@ -361,3 +369,17 @@ def test_instantiate_crs_str_kwargs():
     assert isinstance(crs, ccrs.PlateCarree)
     assert isinstance(crs.globe, ccrs.Globe)
     assert crs.globe.datum == 'WGS84'
+
+
+@pytest.mark.skipif(spd is None or dd is None, reason='spatialpandas or dask is not available')
+def test_is_geodataframe_spatialpandas_dask():
+    square = spd.geometry.Polygon([(0.0, 0), (0, 1), (1, 1), (1, 0)])
+    sdf = spd.GeoDataFrame({'geometry': spd.GeoSeries([square, square]), 'name': ['A', 'B']})
+    sddf = dd.from_pandas(sdf, npartitions=2)
+    assert isinstance(sddf, spd.dask.DaskGeoDataFrame)
+    assert is_geodataframe(sddf)
+
+
+def test_is_geodataframe_classic_dataframe():
+    df = pd.DataFrame({'geometry': [None, None], 'name': ['A', 'B']})
+    assert not is_geodataframe(df)

hvplot/util.py

@@ -474,7 +474,7 @@ def is_geodataframe(data):
     if 'spatialpandas' in sys.modules:
         import spatialpandas as spd
 
-        if isinstance(data, spd.GeoDataFrame):
+        if isinstance(data, (spd.GeoDataFrame, spd.dask.DaskGeoDataFrame)):
             return True
     return (
         isinstance(data, pd.DataFrame) and hasattr(data, 'geom_type') and hasattr(data, 'geometry')

pyproject.toml

@@ -77,6 +77,8 @@ tests = [
     # end fugue
     "ibis-framework[duckdb]",  # ibis-duckdb on conda
     "polars",
+    "dask",
+    "spatialpandas",
 ]
 # In 0.9 fugue added the sql extra but didn't add a fugue-sql package, removing the sql deps from fugue
 # Adding them manually here