-
Notifications
You must be signed in to change notification settings - Fork 19
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
Loosen or upgrade pyarrow dependency #23
Comments
@owenlamont Thanks. I will try to get to it this week. You're right that this started out as just a CLI but then we quickly realized that a library is also useful. |
As a data point: I just ran into this while installing the CLI in my local environment. I've got pyarrow |
@owenlamont @crimeminister I've loosened the dependencies, does that work for your use cases? |
This change worked for me, thanks a lot for jumping on this. |
Cheers! Thank you. |
I was using overturemaps package in this lonboard example which worked fine but I had issues installing it into some Python environments that already used/needed pyarrow 16 for other dependencies.
I'm aware this package is advertised as a CLI tool and not a library so using it as such maybe an abuse of its purpose (and I'd would accept that if it is the intent).
If it is okay to use overturemaps as a library though - it'd be good to keep the dependency versions as loose and wide and possible to accommodate the different environments it could be used in.
The text was updated successfully, but these errors were encountered: