-
Notifications
You must be signed in to change notification settings - Fork 0
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
Interested in PRs? #2
Comments
Also I've been thinking about moving https://github.com/kylebarron/geoarrow-rs to the geoarrow org; would you be interested in moving this or a similar pure-python geoarrow implementation to the geoarrow org? It might be nice to have all the implementations under the same org Edit: I'd forgotten I'd even created a |
Yeah and I can move geoarrow-pandas and geoarrow-pyarrow to geoarrow/geoarrow-python so all the non-native interfaces are all in one place! |
I made geoarrow/geoarrow-python#2 and we can probably discuss there 🙏 |
I think it's important to have efficient and easy to use interop between geoarrow and shapely, and I think this is the best point to start from. Are you interested in any PRs to this package? I.e. are you still interested in maintaining it?
I was talking with @paleolimbot a little while ago and we discussed a namespacing approach to geoarrow in Python. So I could distribute my rust bindings with
geoarrow.rust
and Dewey could distribute his with e.g.geoarrow.c
and they'd be able to interoperate with each other. The main important note here is that there can't be ageoarrow
Python package with an__init__.py
ingeoarrow/__init__.py
because then the namespaced modules will never be found (ref native namespacing docs). So if you do plan to publish this to PyPI, would you be interested in changing the name to be something likegeoarrow.python
orgeoarrow.shapely
(or something else)?In consultation with you and Dewey it might also be a good idea to take the
geoarrow
top level python package name so no one ends up publishing ageoarrow
package that just installs the other namespace packages.The text was updated successfully, but these errors were encountered: