-
-
Notifications
You must be signed in to change notification settings - Fork 401
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
Added Tiles element from geoviews #3515
Conversation
9741fbb
to
d650e4a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me!
Looks good. I didn't know you were working on this so I started working on a PR with the same goal - it looks you beat me to it as I was doing exactly the same thing! |
Happy to see this merged when the tests pass. |
b79a6ac
to
b289a54
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great; happy to see it merged!
Tests are green before the docstring fixes (except for appveyor which passed on the next commit). Merging. |
No description provided.