search

makepath / xarray-spatial

Raster-based Spatial Analytics for Python
https://xarray-spatial.readthedocs.io/
MIT License
828 stars 84 forks source link

natural_breaks(): drop gpu support #705

Closed thuydotm closed 2 years ago

thuydotm commented 2 years ago

The current implementation of natural_breaks algorithm is extremely sequential. The GPU support for this function does not improve but worsen the performance as there is no paralelizable part. It just changes the numpy buffers with cupy buffers without using numba. For these reasons, we should completely remove it.

codecov-commenter commented 2 years ago

Codecov Report

Merging #705 (f20dc7f) into master (56c3ebf) will increase coverage by 0.59%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #705      +/-   ##
==========================================
+ Coverage   80.61%   81.21%   +0.59%     
==========================================
  Files          19       19              
  Lines        4111     4077      -34     
==========================================
- Hits         3314     3311       -3     
+ Misses        797      766      -31
Impacted Files Coverage Δ
xrspatial/classify.py 78.03% <100.00%> (+7.89%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 56c3ebf...f20dc7f. Read the comment docs.