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

Tiles image is not projected if projection is not mercator #529

Closed
MordicusEtCubitus opened this issue Nov 26, 2020 · 2 comments · Fixed by #1053
Closed

Tiles image is not projected if projection is not mercator #529

MordicusEtCubitus opened this issue Nov 26, 2020 · 2 comments · Fixed by #1053
Labels

Comments

@MordicusEtCubitus
Copy link

Hi,
First of all I would like to thanks you a lot for the great work done with HvPlot and the Holoviz ecosystem: plotting huge data with bokeh, cartopy and data shader has never been so easy.

I've tried to plot a big dataframe having 50 millions of records this way:

df is the dataframe

import hvplot.pandas
from cartopy import crs
proj = crs.Robinson() # crs.GOOGLE_MERCATOR  # crs.Mercator()
df.hvplot.points(x='LONGITUDE', y='LATITUDE', geo=True, projection=proj, c='Value'
                 , datashade=True
                 , aggregator=ds.mean('Value')
                 , tiles='ESRI'
                 , coastline=True
                 , project=True
                )

The data and coastlines are correctly projected, but not the background image, as you can see in the attached picture.
Is this a bug or did I missed something ?
Thanks a lot for helping !

import cartopy
print(hvplot.__version__)
print(cartopy.__version__)

0.6.0
0.18.0

robinson_hvplot

@jbednar
Copy link
Member

jbednar commented Nov 27, 2020

I would have expected at least a warning in that case, if not an error, because tiles only support a Web Mercator projection. I would have hoped that everything gets forced into Web Mercator, with a warning saying that the projection is being ignored, or else an error saying that tiles cannot be overlaid because the projection is not Web Mercator.

@maximlt maximlt added the geo label Oct 25, 2022
@jsignell
Copy link
Member

jsignell commented Apr 4, 2023

I will include this in my PR tackling #951 and #738

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants