-
Notifications
You must be signed in to change notification settings - Fork 18
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
Keep/add or deprecate GeoJSON input type? #343
Comments
The more I'm thinking about this the more I'm leaning towards option 2... |
yes, imagine that everywhere where you can now pass a raster cube, you would also be able to pass a multi-dimensional array construct like |
Also, there are not a lot of processes affected by the deprecation. I'm only sure we'd deprecate in aggregate_spatial. I'm not sure about filter_spatial and mask_polygon. |
By the way, for UC8 I think that we'll use the old geoJSON spec, which allows different projections specifying the "crs" value. This can be replaced by a load_vector_cube process once it will be ready. |
I'll try to draft a process soon as this would indeed make other process descriptions easier. |
Draft proposal available in PR #412. |
With the introduction of vector data cubes there's the question whether the geojson type should still be used and added in vector processes or not.
Options:
I'm not sure what I prefer. In processes, it could be easier to not have it, in (my) tooling/implementations it is probably a bit easier to have it.
Origin: https://github.com/Open-EO/openeo-processes/pull/315/files#r814788548
Closely related to #323
The text was updated successfully, but these errors were encountered: