Kelsey Jordahl kjordahl

Organizations

@enthought @scipy-conference @geopandas
@kjordahl
kjordahl commented on issue kjordahl/osm-usfs#3
@kjordahl

Fixed many more than 7 self-intersections, still getting 3 "Intersection between multipolygon ways" warnings.

@kjordahl
  • @kjordahl e345f58
    Simplify and fix warnings for Kootenai NF
kjordahl commented on issue geopandas/geopandas#196
@kjordahl

Yes, you would need to convert your timestamp values to strings. GeoPandas is very flexible about what kinds of data you can store in columns, but …

kjordahl commented on issue geopandas/geopandas#195
@kjordahl

Hmm, I guess this way of adding a new column should work (and it does for Polygons or non-geometry columns). It is a bit unusual, though. The prefe…

@kjordahl
  • @kjordahl c2c43b6
    Simplify and tag Salmo-Priest Wilderness for upload
kjordahl deleted branch sqlalchemy at geopandas/geopandas
kjordahl deleted branch feature/support-null-geometries at geopandas/geopandas
kjordahl deleted branch bug/sjoin-index at geopandas/geopandas
kjordahl deleted branch clean/add-kwarg-null-value at geopandas/geopandas
@kjordahl
  • @kjordahl 6163bc9
    Merge pull request #194 from geopandas/clean/add-kwarg-null-value
  • @kjordahl bc3dc90
    CLN: Add kwarg for calls to _series_unary_op() even when using defaul…
  • 1 more commit »
kjordahl merged pull request geopandas/geopandas#194
@kjordahl
Always use kwargs when calling _series_unary_op()
2 commits with 7 additions and 7 deletions
kjordahl opened pull request geopandas/geopandas#194
@kjordahl
Always use kwargs when calling _series_unary_op()
2 commits with 7 additions and 7 deletions
@kjordahl
kjordahl merged pull request geopandas/geopandas#193
@kjordahl
Handle missing geoms
9 commits with 77 additions and 31 deletions
kjordahl commented on pull request geopandas/geopandas#193
@kjordahl

I'm going to merge this and follow up on my nits above. Thanks @lucasnad27 and @IamJeffG! Sorry this one fell off the radar for so long, good to ha…

@kjordahl
Upload to OpenStreetMaps
@kjordahl
  • @kjordahl 189c641
    Simplify and fix validation errors for IPNF
kjordahl commented on pull request geopandas/geopandas#193
@kjordahl

Looks good to me with minor comments. This change handles a shapefile with null geometries that I found in the wild: zipfile. Nice!

kjordahl commented on pull request geopandas/geopandas#193
@kjordahl

It might be cleaner to put this in the tests directory (e.g. tests/data) instead. On the other hand, this is where we download the NY boroughs shap…

kjordahl commented on pull request geopandas/geopandas#193
@kjordahl

I'd prefer to call this as a kwarg: null_value=np.nan.

kjordahl closed pull request geopandas/geopandas#156
@kjordahl
Include support for null geometries
kjordahl commented on pull request geopandas/geopandas#156
@kjordahl

Closing in favor of #193

@kjordahl
Problems with new spatial join
kjordahl commented on issue geopandas/geopandas#190
@kjordahl

Closed by #191.

@kjordahl
kjordahl merged pull request geopandas/geopandas#191
@kjordahl
Fix bug in sjoin index
7 commits with 15 additions and 18 deletions
kjordahl commented on pull request lucasnad27/geopandas#1
@kjordahl

@IamJeffG Thanks for doing this. Can you make this PR against the main geopandas repository so we can discuss it there? I'd like to get this merged.