Open-EO / openeo-processes-python

A Python representation of (most) openEO processes
Apache License 2.0
11 stars 4 forks source link

Return lazy geodataframe from load_vector_cube #149

Closed LukeWeidenwalker closed 2 years ago

LukeWeidenwalker commented 2 years ago

I'm a bit torn about this change - dask_geopandas can't actually load the files, so they'll be loaded into memory here regardless of this change. I just thought it might be easier to just stick to this lazy datatype, so that later functions don't always have to check whether something is a geopandas or a dask_geopandas dataframe. Any thoughts on this?

clausmichele commented 2 years ago

Seems reasonable. I can't think of scenarios where dask_geopandas would not work at the moment, we will see in the future how we manage the new vector processes with it.