DimensionalData.jl icon indicating copy to clipboard operation
DimensionalData.jl copied to clipboard

Add a DiskArrays extension

Open rafaqz opened this issue 1 year ago • 1 comments

As wrapping some kind of DiskArray is a major use case of DimensionalData we may as well move the diskarrays forwarding from Rasters/YAXArrays/PyramidScheme to an extension here, and then they will work in any other wrapper too.

This will mean we can do proper testing of broadcasting and iteration over wrapped DiskArrays here rather than in the extending packages.

For motivation see: https://github.com/meggart/DiskArrays.jl/issues/185

@felixcremer @meggart does that make sense to you? Things are how they are just because extensions didn't exist before, if they did it probably would have been put here from the start.

rafaqz avatar Aug 18 '24 11:08 rafaqz

Yes, I think it would make sense to streamline this and make the interaction clearer between Dimensions and DiskArrays.

felixcremer avatar Aug 26 '24 09:08 felixcremer

We have this now

rafaqz avatar Jan 30 '25 20:01 rafaqz

I think we still don't forward a bunch of stuff (isdisk etc). Should we do that?

asinghvi17 avatar Jan 30 '25 20:01 asinghvi17

isdisk was only added to DiskArrays last week ;)

But forward away, that's why I added it

rafaqz avatar Jan 31 '25 08:01 rafaqz