Skip to content

Added a minimum working example for the highlight_function #630

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

Merged
merged 2 commits into from
Jun 24, 2017

Conversation

eHanseJoerg
Copy link
Contributor

I added a Jupyter Notebook and some csv data to show how the highlight function works.
The Jupyter NB depends on pandas, os, folium, json.

Additionally, I added two lines to the docstring in the GeoJson class because the paramter list was incomplete. I did not work on chloropleth class.

… in the GeoJson class.

This example shows how the style of an object on the map changes when the mouse is being moved over it.
@ocefpaf
Copy link
Member

ocefpaf commented Jun 15, 2017

Thanks @eHanseJoerg!

GitHub makes it hard to review notebooks. So here is a nbviewer link to hlelp: http://nbviewer.jupyter.org/github/eHanseJoerg/folium/blob/master/examples/Highlight_Function.ipynb

@joshuacano
Copy link
Contributor

joshuacano commented Jun 16, 2017 via email

@ocefpaf ocefpaf merged commit b4445e7 into python-visualization:master Jun 24, 2017
@ocefpaf
Copy link
Member

ocefpaf commented Jun 24, 2017

Thanks @eHanseJoerg!

@elochk
Copy link

elochk commented Sep 23, 2017

Thanks for the example! I am trying to plot polygons in the same way, but nothing displays at the map. When I change type "Polygon" to type "LineString" keeping the same coordinates, they are displayed. I would be grateful if you could help with this issue.

@amirham
Copy link

amirham commented Oct 17, 2017

In the same situation as @elochk, this doesn't seem to work for Polygons.

sanga pushed a commit to sanga/folium that referenced this pull request Oct 24, 2017
Added a minimum working example for the highlight_function
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.

5 participants