-
Notifications
You must be signed in to change notification settings - Fork 57
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
Update scripts dependencies #179
Conversation
This fails the tests because it uses a later version of GeoPandas/Pyproj and writes PROJJSON with a schema of |
This should probably be a dedicated ticket, but I've run into this same problem that the CI job is hitting with the projjson schema dependency. It is not a real problem until a version of projjson is released that is incompatible with https://proj.org/schemas/v0.5/projjson.schema.json. But it looks like we need to relax the |
It looks like pytest actually succeeds (you can run geoparquet/.github/workflows/scripts.yml Lines 57 to 61 in ad32cca
|
Just saw the same after re-reading. Yes, |
Woohoo. Thanks, @kylebarron |
I'll merge this so we can proceed with the re-release. Hope that sounds ok. Thanks for fixing it up. |
Closes #178
Now
works on my machine