xarray-spatial icon indicating copy to clipboard operation
xarray-spatial copied to clipboard

Enable dask+cupy (cuda) implementation for (some) convolution methods, e.g. aspect

Open relativityhd opened this issue 1 month ago • 2 comments

Fixes #814 Fixes #140

Proposed Changes

  • Enables Dask + Cupy for aspect
  • Enables Dask + Cupy for convolve2d
  • Enables Dask + Cupy for curvature
  • Enables Dask + Cupy for slope

relativityhd avatar Dec 03 '25 13:12 relativityhd

I already used these implementations for a project of mine, and they seemed to work fine. However, I did not extensive testing or encountered edges yet.

Should I try to enable dask + cupy for other functions as well?

relativityhd avatar Dec 03 '25 14:12 relativityhd

I already used these implementations for a project of mine, and they seemed to work fine. However, I did not extensive testing or encountered edges yet.

Should I try to enable dask + cupy for other functions as well?

@relativityhd thanks so much for the quick response. What you have here looks good and we'll need to get some test which can run that code path along with updating docs. I should have some time to help later and test on my local environment.

I believe that this would be related to the old...but open issue here: https://github.com/makepath/xarray-spatial/issues/140

brendancol avatar Dec 03 '25 16:12 brendancol