nasa / opera-sds-pcm

Observational Products for End-Users from Remote Sensing Analysis (OPERA)
Apache License 2.0
16 stars 12 forks source link

[New Feature]: geo_util.py::does_bbox_intersect_region can be modified to accept geojson with overlapping features #816

Open philipjyoon opened 5 months ago

philipjyoon commented 5 months ago

Checked for duplicates

Yes - I've already checked

Alternatives considered

Yes - and alternatives don't suffice

Related problems

nt

Describe the feature request

When using geojson with overlapping features in geofiltering in the query job, you often get error from the underlying library which is documented in this ticket: https://github.com/nasa/opera-sds-pcm/issues/815

This function in conjunction with _load_region_opera_geometry_collection can be slightly modified to check for one feature at a time and that seems to allow for geojsons with overlapping features. We're not sure whether such geojson is even valid but if we decide to support such files in the future, it's an easy change between these two functions.

We would want to perform some performance testing to make sure that it doesn't bog down the function.