-
Notifications
You must be signed in to change notification settings - Fork 4
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
support generic reducers in aggregate_spatial (aka zonal_statistics) #49
Comments
also see Open-EO/openeo-python-client#144 |
Note that this is far from trivial, the current code path is based on computing histograms, and can only support statistics based on that. |
Indeed, generic However there are still some simple one-process reducers we could easily add (based on histograms),
|
this one got fixed, we now also support using multiple reducers. |
aggregate_spatial
/zonal_statistics
in openeo-geopyspark-driver (and openeo-python-driver) assumes the provided reducer is a simple one-process reducer, and only supports: 'histogram', 'sd', 'median' or 'mean'(also: I think it would be cleaner if we could also remove the usage of old-style "zonal_statistics" and use 1.0-style "aggregate_spatial" in more places)
The text was updated successfully, but these errors were encountered: