geoarrow / geoarrow-python

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

Fix double import in `geoarrow.pyarrow` resulting in double registering extension types #6

Closed paleolimbot closed 1 year ago

kylebarron commented 1 year ago

At least it's reproducible! A simple workaround is just to re-register the extension type whenever you get this error (unregister then register). That's what I did in the pure python version

codecov[bot] commented 1 year ago

Codecov Report

Merging #6 (1a17afe) into main (bea9ae3) will not change coverage. The diff coverage is 100.00%.

:exclamation: Current head 1a17afe differs from pull request most recent head fc08709. Consider uploading reports for the commit fc08709 to get more accurate results

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   94.60%   94.60%           
=======================================
  Files           9        9           
  Lines        1241     1241           
=======================================
  Hits         1174     1174           
  Misses         67       67           
Files Changed Coverage Δ
geoarrow-pyarrow/src/geoarrow/pyarrow/__init__.py 100.00% <100.00%> (ø)
geoarrow-pyarrow/src/geoarrow/pyarrow/_array.py 90.37% <100.00%> (ø)
geoarrow-pyarrow/src/geoarrow/pyarrow/_compute.py 97.94% <100.00%> (ø)
geoarrow-pyarrow/src/geoarrow/pyarrow/_dataset.py 99.52% <100.00%> (ø)
geoarrow-pyarrow/src/geoarrow/pyarrow/_kernel.py 94.18% <100.00%> (ø)
geoarrow-pyarrow/src/geoarrow/pyarrow/_scalar.py 63.88% <100.00%> (ø)