openeo-python-client icon indicating copy to clipboard operation
openeo-python-client copied to clipboard

Eliminate `XarrayDataCube`?

Open soxofaan opened this issue 2 years ago • 2 comments

XarrayDataCube is extremely simple: it just wraps a single xarray.DataArray and nothing more. It offers some IO functionality, but that doesn't mean these have to be XarrayDataCube methods. (There is also a plot method, but I wonder if anybody uses that and I dislike that this drags in a dependency on matplotlib)

It's simple, but it creates an extra abstraction layer in working with UDF, possibly making things more confusing than necessary.

can we just drop (the need for) XarrayDataCube ?

soxofaan avatar Sep 11 '23 15:09 soxofaan

This is there on backend side, we will of course have to keep XArrayDataCube around for quite a bit. Next step is to update documentation to favour UDF's without XArrayDataCube in the signature.

jdries avatar Dec 20 '23 08:12 jdries

Indeed, we have to keep XarrayDataCube for legacy reasons and backward compatibility. My questions/proposal is mainly about having a new, simpler UDF signature API

soxofaan avatar Jan 04 '24 14:01 soxofaan