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

Use `DiskArrays.pad` in `extend` when the parent `isdisk`

Open rafaqz opened this issue 9 months ago • 3 comments
trafficstars

Waiting on https://github.com/JuliaIO/DiskArrays.jl/pull/219/

rafaqz avatar Feb 19 '25 16:02 rafaqz

Or possibly add a DiskArrays.pad method as instead? extend doesn't specify a padding value and assumes a writable array is returned.

pad would assume the exact semantics of DiskArrays.pad, and it would work without supplying new dimensions as long as the extended dims are Regular

rafaqz avatar Feb 19 '25 16:02 rafaqz

Extending pad sounds better for the specific use case you want. But we'll have to be very clear when documenting this and PaddedDiskArray should have a nice error when you try to write to it.

asinghvi17 avatar Feb 19 '25 16:02 asinghvi17

Yeah, it already has an error message in writeblock!

rafaqz avatar Feb 19 '25 16:02 rafaqz