disallow Points

CHANGELOG.md

@@ -23,12 +23,15 @@ and start a new "In Progress" section above it.
 
 - array_apply: sub-process should now work on all supported processes ([Open-EO/openeo-geopyspark-driver#1064](https://github.com/Open-EO/openeo-geopyspark-driver/issues/1064))
 
+## 0.128.0
+
+- `load_collection`/`load_stac`: `spatial_extent` requires (Multi)Polygon geometries ([Open-EO/openeo-geopyspark-driver#996](https://github.com/Open-EO/openeo-geopyspark-driver/issues/996))
+
 ## 0.127.0
 
 - Add `simple_job_progress_estimation` config for simple job progress estimation ([Open-EO/openeo-geopyspark-driver#772](https://github.com/Open-EO/openeo-geopyspark-driver/issues/772))
 - `OpenEoBackendConfig`: be more forgiving about unknown config keys to better support use cases that involve backward/forward incompatible configurations ([#322](https://github.com/Open-EO/openeo-python-driver/issues/322))
 
-
 ## 0.126.0
 
 - Add STAC collections conformance class ([#195](https://github.com/Open-EO/openeo-python-driver/issues/195))

openeo_driver/ProcessGraphDeserializer.py

@@ -1461,13 +1461,21 @@ def filter_labels(args: ProcessArgs, env: EvalEnv) -> DriverDataCube:
 
 def _extract_bbox_extent(args: dict, field="extent", process_id="filter_bbox", handle_geojson=False) -> dict:
     extent = extract_arg(args, name=field, process_id=process_id)
+    # TODO #114: support vector cube
     if handle_geojson and extent.get("type") in [
         "Polygon",
         "MultiPolygon",
-        "GeometryCollection",
+        "GeometryCollection",  # TODO #71 #114: deprecate GeometryCollection
         "Feature",
         "FeatureCollection",
     ]:
+        if not _contains_only_polygons(extent):
+            raise ProcessParameterInvalidException(
+                parameter=field,
+                process=process_id,
+                reason="unsupported GeoJSON; requires at least one Polygon or MultiPolygon",
+            )
+
         w, s, e, n = DriverVectorCube.from_geojson(extent).get_bounding_box()
         # TODO: support (non-standard) CRS field in GeoJSON?
         d = {"west": w, "south": s, "east": e, "north": n, "crs": "EPSG:4326"}
@@ -1483,6 +1491,19 @@ def _extract_bbox_extent(args: dict, field="extent", process_id="filter_bbox", h
     return d
 
 
+def _contains_only_polygons(geojson: dict) -> bool:
+    if geojson["type"] in ["Polygon", "MultiPolygon"]:
+        return True
+
+    if geojson["type"] == "Feature":
+        return _contains_only_polygons(geojson["geometry"])
+
+    if geojson["type"] == "FeatureCollection":
+        return all(_contains_only_polygons(feature) for feature in geojson["features"])
+
+    return False
+
+
 @process
 def filter_bbox(args: ProcessArgs, env: EvalEnv) -> DriverDataCube:
     cube: DriverDataCube = args.get_required("data", expected_type=DriverDataCube)

openeo_driver/_version.py

@@ -1 +1 @@
-__version__ = "0.127.0a1"
+__version__ = "0.128.0a1"

tests/test_dry_run.py

@@ -1,3 +1,4 @@
+from contextlib import nullcontext
 from pathlib import Path
 from unittest import mock
 
@@ -18,7 +19,7 @@
     ProcessType,
 )
 from openeo_driver.dummy.dummy_backend import DummyVectorCube
-from openeo_driver.errors import OpenEOApiException
+from openeo_driver.errors import OpenEOApiException, ProcessParameterInvalidException
 from openeo_driver.ProcessGraphDeserializer import (
     ENV_DRY_RUN_TRACER,
     ENV_MAX_BUFFER,
@@ -1447,6 +1448,74 @@ def test_load_stac_properties(dry_run_env, dry_run_tracer):
     ]
 
 
+@pytest.mark.parametrize(
+    ["spatial_extent", "expectation"],
+    [
+        ({"type": "Polygon", "coordinates": [[[0, 0], [1, 1], [1, 0]]]}, nullcontext()),
+        (
+            {
+                "type": "Feature",
+                "geometry": {"type": "MultiPolygon", "coordinates": [[[[0, 0], [1, 1], [1, 0]]]]},
+                "properties": {},
+            },
+            nullcontext(),
+        ),
+        (
+            {
+                "type": "Feature",
+                "geometry": {"type": "Point", "coordinates": [5, 50]},
+                "properties": {},
+            },
+            pytest.raises(
+                ProcessParameterInvalidException,
+                match=r"unsupported GeoJSON; requires at least one Polygon or MultiPolygon$",
+            ),
+        ),
+        (
+            {
+                "type": "FeatureCollection",
+                "features": [
+                    {
+                        "type": "Feature",
+                        "geometry": {"type": "Point", "coordinates": [5, 50]},
+                        "properties": {},
+                    },
+                    {
+                        "type": "Feature",
+                        "geometry": {"type": "Polygon", "coordinates": [[[0, 0], [1, 1], [1, 0]]]},
+                        "properties": {},
+                    },
+                ],
+            },
+            pytest.raises(
+                ProcessParameterInvalidException,
+                match=r"unsupported GeoJSON; requires at least one Polygon or MultiPolygon$",
+            ),
+        ),
+    ],
+)
+def test_load_stac_spatial_extent_requires_a_polygon(
+    dry_run_tracer, backend_implementation, spatial_extent, expectation
+):
+    pg = {
+        "loadstac1": {
+            "process_id": "load_stac",
+            "arguments": {
+                "url": "https://stac.test",
+                "spatial_extent": spatial_extent,
+            },
+            "result": True,
+        }
+    }
+
+    dry_run_env = EvalEnv(
+        {ENV_DRY_RUN_TRACER: dry_run_tracer, "backend_implementation": backend_implementation, "version": "2.0.0"}
+    )
+
+    with expectation:
+        evaluate(pg, dry_run_env)
+
+
 @pytest.mark.parametrize(
     ["arguments", "expected"],
     [

tests/test_views_execute.py

@@ -353,13 +353,6 @@ def test_load_collection_filter(api):
             ),
             {"west": 11, "south": 22, "east": 44, "north": 55, "crs": "EPSG:4326"},
         ),
-        (
-            as_geojson_feature_collection(
-                shapely.geometry.Point(2, 3),
-                shapely.geometry.Point(4, 5),
-            ),
-            {"west": 2, "south": 3, "east": 4, "north": 5, "crs": "EPSG:4326"},
-        ),
     ],
 )
 def test_load_collection_spatial_extent_geojson(api, spatial_extent, expected):