Skip to content

Conversation

@fscottfoti
Copy link
Contributor

basically we monkey patch geopandas to have an explore method that calls dframe_explorer with all the right configuration.

This is nice because geopandas

  • can convert to 4326
  • has the center of the shapefile already
  • can read shapefiles rather than geojson into a leaflet map
  • can convert to geojson on the fly so that no file on the filesystem is required

This also calls the webbrowser automatically when everything is ready

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.18%) when pulling 44d350e on geopandas-explorer into 0b3cf9c on master.

setup.py Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Think we should leave this as an optional dependency so it doesn't gum people up who don't plan to use it?

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 20bec6b on geopandas-explorer into * on master*.

basically we monkey patch geopandas to have an explore method that calls dframe_explorer with all the right configuration.

This is nice because geopandas
* can convert to 4326
* has the center of the shapefile already
* can read shapefiles rather than geojson into a leaflet map
* can convert to geojson on the fly so that no file on the filesystem is required

This also calls the webbrowser automatically when everything is ready
…_explorer

still needs a test for this new function
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.95%) when pulling 8ea9c22 on geopandas-explorer into e5e2037 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.28%) when pulling b6aa0fd on geopandas-explorer into e5e2037 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants