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

add required attribution for explores using CartoDB darkmatter tiles #73

Closed
wants to merge 1 commit into from

Conversation

arjunbanker
Copy link

Currently, executing many of the examples in "11-interactive-web-mapping notebook with required attribution" yields
this error: "ValueError: Custom tiles must have an attribution."

This diff resolves the error by adding the required "attr" param to the explore calls.

@gboeing
Copy link
Owner

gboeing commented Sep 17, 2023

@arjunbanker I'm not seeing this error with GeoPandas 0.13.x or 0.14.x. Can you provide a complete list of your package versions (such as with conda list) and the complete error traceback? Thanks.

@arjunbanker
Copy link
Author

arjunbanker commented Sep 18, 2023 via email

@gboeing
Copy link
Owner

gboeing commented Sep 19, 2023

I'm still unable to reproduce this error with OSMnx 1.6.0, GeoPandas 0.13.2 or 0.14.0, and Folium 0.14.0. If this is a conflict with a recent version of one of the dependencies, I cannot see where it comes from. Can you try another environment to see if you can reproduce it elsewhere? For example, try Binder.

@arjunbanker
Copy link
Author

appreciate the link to Binder! the issue doesn't seem to reproduce there, so it was likely some local dependency issue on my end. closing this out

@arjunbanker arjunbanker closed this Oct 6, 2023
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.

2 participants