geoarrow / geoarrow-python

Python implementation of the GeoArrow specification
http://geoarrow.org/geoarrow-python/
Apache License 2.0
59 stars 3 forks source link

refactor: Remove threaded compute function options #20

Closed paleolimbot closed 11 months ago

paleolimbot commented 11 months ago

This was a half-baked part of the initial code that was never actually used. Concurrency should almost certainly be handled above (e.g., dask) or below (e.g., in geoarrow-c)...what ended up happening was that the tasks took so little time to complete that it didn't end up helping much.

codecov[bot] commented 11 months ago

Codecov Report

Merging #20 (054c3fb) into main (25c567c) will decrease coverage by 0.03%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   94.53%   94.50%   -0.03%     
==========================================
  Files           9        9              
  Lines        1245     1220      -25     
==========================================
- Hits         1177     1153      -24     
+ Misses         68       67       -1     
Files Coverage Δ
geoarrow-pyarrow/src/geoarrow/pyarrow/_compute.py 97.80% <100.00%> (+0.21%) :arrow_up: