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

raster <-> vector processes #155

Open
jdries opened this issue Jun 9, 2020 · 10 comments
Open

raster <-> vector processes #155

jdries opened this issue Jun 9, 2020 · 10 comments
Assignees
Labels
help wanted Extra attention is needed new process vector
Milestone

Comments

@jdries
Copy link
Contributor

jdries commented Jun 9, 2020

For an internal use case, we'd like to apply raster to vector conversion:

  1. input is a raster datacube
  2. output is a 'vector datacube' that can be exported as geojson

This is not uncommon, so we'd like to see if we can provide this as higher level process.
We will probably implement something internally first, to see how it can look like.

@m-mohr
Copy link
Member

m-mohr commented Jun 9, 2020

As soon as you have a process definition I'd suggest to open a draft PR so that we can shape it together.

@m-mohr m-mohr added new process help wanted Extra attention is needed labels Jun 9, 2020
@jdries
Copy link
Contributor Author

jdries commented Jun 9, 2020

Some inspiration for naming:

@jdries jdries changed the title segmentation process raster to vector process Jun 9, 2020
@m-mohr
Copy link
Member

m-mohr commented Jun 9, 2020

Can we model this as reducer in openEO? If not, reduce would be a bit confusing, I think. polygonize only works if it's always working on polygons only. I think I'd prefer something easy to understand and aligned with other processes such as create_raster_cube like vector_cube, to_vector or convert_cube (if it also allows vector to raster at some point). I'm not set on anything yet. Question is also how we'd like to name vector to raster once it comes up?

@jdries
Copy link
Contributor Author

jdries commented Jun 10, 2020

Implemented a first prototype:
https://openeo.vito.be/openeo/1.0/processes/backend/raster_to_vector

@m-mohr
Copy link
Member

m-mohr commented Jun 11, 2020

@jdries We should complete the description and PR it for discussion.

@m-mohr m-mohr modified the milestones: v1.0-final, future Jun 29, 2020
@m-mohr m-mohr added the minor label Aug 4, 2020
@m-mohr m-mohr changed the title raster to vector process raster <-> vector processes Apr 12, 2021
@m-mohr m-mohr added the vector label Apr 21, 2021
@m-mohr m-mohr mentioned this issue Dec 17, 2021
@m-mohr m-mohr removed the minor label Apr 4, 2023
@PondiB PondiB closed this as completed Apr 19, 2023
@PondiB PondiB reopened this Apr 19, 2023
@PondiB
Copy link
Member

PondiB commented Apr 19, 2023

@jdries and @m-mohr, is this being worked on?

@m-mohr
Copy link
Member

m-mohr commented Apr 19, 2023

VITO has a custom implementation, but we still need to get a version into openeo-processes. @PondiB

@PondiB
Copy link
Member

PondiB commented Apr 19, 2023

Cool, I will pick it up then.

@PondiB PondiB self-assigned this Apr 19, 2023
@m-mohr
Copy link
Member

m-mohr commented Apr 19, 2023

Cool, please consider a survey of existing implementations, e.g. at VITO (already openEO), GRASSGIS, Python ecosystem (shapely?), R, ... @PondiB

@PondiB
Copy link
Member

PondiB commented Apr 19, 2023

@m-mohr , Sure, I will do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed new process vector
Projects
None yet
Development

No branches or pull requests

3 participants