Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate removing libspatialindex dependency #36

Closed
lmnoel opened this issue Feb 10, 2019 · 4 comments
Closed

Investigate removing libspatialindex dependency #36

lmnoel opened this issue Feb 10, 2019 · 4 comments
Assignees

Comments

@lmnoel
Copy link
Collaborator

lmnoel commented Feb 10, 2019

geopandas spatial join requires installation of libspatialindex by brew/apt-get. Try to find an alternative implementation of spatial joins to simplify installation

@lmnoel lmnoel self-assigned this Feb 10, 2019
@jkoschinsky
Copy link
Collaborator

jkoschinsky commented Feb 10, 2019 via email

@lmnoel
Copy link
Collaborator Author

lmnoel commented Feb 10, 2019

Unfortunately I don't think that would help--the problem is that my current solution requires installing a library with a system package manager (aka outside of python)--relatively much more complicated than pip installing. Boost and GDAL have the same problem, as far as I know--although please let me know if I'm wrong

@lixun910
Copy link
Member

@lmnoel I see you are dynamic linking protobuf library, which is also required to be installed outside of python. Can you install libspatialindex with libprotobuf?

Another way is to implement spatial join from scrach if you only do points-n-polygon spatial joining: there are many headers only R-tree for spatial join, and you just need to add points-n-polygon detection code.

@lmnoel
Copy link
Collaborator Author

lmnoel commented Feb 12, 2019

@lixun910 Actually in the process of removing protobuf right now, trying to eliminate the need for all non-standard libraries.

I will definitely look in to implementing it from scratch if nothing better comes up

@lmnoel lmnoel closed this as completed Apr 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants