Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Empty geometry iso error for vector_buffer with negative buffer on small geometries #238

Open
soxofaan opened this issue Nov 30, 2023 · 3 comments
Assignees

Comments

@soxofaan
Copy link
Member

Open-EO/openeo-processes#404 added specification what should happen when negative buffering of small geometries with vector_buffer: use empty geometries.

Current implementation in openeo-python-driver however throws an error. Originally it was an obscure error message (see Open-EO/openeo-geopyspark-driver#314). The error was improved in #164 as quick fix.

Next step is to introduce empty geometries (and support them in subsequent processes like aggregate_spatial) instead of throwing errors.

@soxofaan
Copy link
Member Author

(internal reference: MKTP-256)

@jdries
Copy link
Contributor

jdries commented Dec 5, 2023

fine for me, doesn't sound too hard

@soxofaan
Copy link
Member Author

soxofaan commented Dec 5, 2023

the most annoying part will probably be to make sure that all the downstream processes like aggregate_spatial, mask_polygon, apply_polygon, ... don't choke on these new empty geometries.

@jdries jdries self-assigned this Dec 14, 2023
jdries added a commit to Open-EO/openeo-geotrellis-extensions that referenced this issue Dec 14, 2023
@jdries jdries assigned EmileSonneveld and unassigned jdries Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants