roocs / clisops

Climate Simulation Operations
https://clisops.readthedocs.io/en/latest/
Other
21 stars 9 forks source link

shape_bbox_indexer seems fragile with some geometries #280

Closed huard closed 1 year ago

huard commented 1 year ago

Description

When using clisops.core.average_shape, I'm getting GEOSException: TopologyException: unable to assign free hole to a shell with a multipolygon. This error is raised by shape_bbox_indexer unary_union operation.

The logic in shape_bbox_indexer is only meant to make a rough subset of the area of interest to reduce data transfers. We can probably compute a simple convex hull to get the rough shape of the bounding box.

huard commented 1 year ago

Closed with #282