-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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: