-
Notifications
You must be signed in to change notification settings - Fork 34
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
Integrate mapclassify
#136
Comments
The following worked for me:
|
Thanks! By the way, I'm not sure where Viridis_20.colors[jc.yb] |
since this is still open, i've cooked up a proof of concept that uses pandas's extension API to create a geopandas works well for my use cases and I'm happy to keep it in geosnap if it's out of scope. That said, I suspect it might be more useful/visible in a central location like lonboard itself so also more than happy to contribute it if anyone has interest |
heres a little notebook that updates the color scheme dynamically using jupyterwidgets (note, not all classifiers take advantage of all https://gist.github.com/knaaptime/cc6863d99a4a08a9825191dc27b3a5f1 |
Thanks @knaaptime, that's very cool. I don't think I want to add from lonboard.geopandas import register_plot_methods
register_plot_methods() Then gdf = gpd.GeoDataFrame(..)
gdf.plot()
# or maybe
gdf.lb.explore()
# or even have a way to override .explore Would you be open to making the PR? Then we could have the mapclassify code be self-contained within the GeoPandas integration. |
cool, that's exactly what i was thinking (agree, no desire to make mapclassify a hard dependency). Happy to put together the PR |
i'll start the PR so the code is more visible, but for some reason I cant import the local version of lonboard because it cant find |
(nvm, i just grabbed those files from the tarball on pypi; looks like maybe they never got committed or something) |
From the geopandas.plot docstring
The text was updated successfully, but these errors were encountered: