geoarrow / geoarrow-python

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

Unpin pyarrow dependency #25

Closed paleolimbot closed 11 months ago

paleolimbot commented 11 months ago

...this was happening because the extension type defined .id, and a recent update to pyarrow actually checks .id from the Python side!

codecov[bot] commented 11 months ago

Codecov Report

Merging #25 (7c4a7fb) into main (0d7657c) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 7c4a7fb differs from pull request most recent head fd8799f. Consider uploading reports for the commit fd8799f to get more accurate results

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   94.25%   94.25%           
=======================================
  Files          10       10           
  Lines        1219     1219           
=======================================
  Hits         1149     1149           
  Misses         70       70           
Files Coverage Δ
geoarrow-pandas/src/geoarrow/pandas/lib.py 93.50% <100.00%> (ø)
geoarrow-pyarrow/src/geoarrow/pyarrow/_compute.py 97.80% <100.00%> (ø)
geoarrow-pyarrow/src/geoarrow/pyarrow/_type.py 95.49% <100.00%> (ø)