Skip to content
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

Geopandas explorer #108

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Geopandas explorer #108

wants to merge 11 commits into from

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.

@@ -22,6 +22,7 @@
packages=find_packages(exclude=['*.tests']),
install_requires=[
'bottle>=0.12.5',
'geopandas>=0.1.0',
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.

3 participants