-
Notifications
You must be signed in to change notification settings - Fork 821
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
better handling of null geometry column #1012
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1012 +/- ##
==========================================
- Coverage 98.04% 98.04% -0.01%
==========================================
Files 23 23
Lines 2409 2407 -2
==========================================
- Hits 2362 2360 -2
Misses 47 47 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @martinfleis I appreciate you taking a look. Tests failed silently (due to a bug fixed by #1014). I made a change to fix the linter and proposed a change to fix the tests. Let me know what you think.
Fixes #1006
This should result in the same GeoDataFrame without intermediate steps which behaviour will change in future.