reproject icon indicating copy to clipboard operation
reproject copied to clipboard

Find a way to avoid customizing map_coordinates

Open astrofrog opened this issue 3 years ago • 2 comments

We have a wrapper of map_coordinates in array_utils.py which pads the array before being called - however this causes the input array to be loaded into memory, whereas if we didn't do this map_coordinates from scipy could deal well with memory-mapped arrays.

We should explore this again and see if we can avoid having to include this wrapper.

astrofrog avatar Feb 28 '23 15:02 astrofrog

Also presumably dropping the wrapper would help with the future use of any dask-image variant of map_coordinates?

Cadair avatar Mar 01 '23 12:03 Cadair

Indeed!

astrofrog avatar Mar 01 '23 14:03 astrofrog

We're going to have to use the wrapper for the foreseeable future as it also now handles the fact that the default map_coordinates loads the whole array for big-endian data. However, we no longer have memory issues in ours, so we can close this now.

astrofrog avatar Jun 13 '24 10:06 astrofrog