holoviz / holoviews

With Holoviews, your data visualizes itself.
https://holoviews.org
BSD 3-Clause "New" or "Revised" License
2.7k stars 402 forks source link

Hover showing lon/lat in meters instead of degrees #3820

Closed rsignell-usgs closed 5 years ago

rsignell-usgs commented 5 years ago

The simple reproducible code below used to display on the lon/lat correctly in degrees, but now is displaying in meters:

import xarray as xr
import hvplot.xarray
import cartopy.crs as ccrs
import geoviews as gv

url = 'http://thredds.ucar.edu/thredds/dodsC/grib/NCEP/WW3/Global/Best'
ds = xr.open_dataset(url)
mesh = ds.Wind_speed_surface[1,:,:].hvplot(rasterize=True, crs=ccrs.PlateCarree())
gv.tile_sources.OSM * mesh

holoviews

Is this a bug, an environment issue or a user error?

rsignell-usgs commented 5 years ago

This even simpler example with the xarray test data also has the problem:

import hvplot.xarray
import geoviews as gv
import cartopy.crs as ccrs
import xarray as xr

ds = xr.tutorial.open_dataset('rasm').load()
ds.Tair[0,:,:].hvplot.quadmesh(x='xc', y='yc', rasterize=True, crs=ccrs.PlateCarree())

produces: 2019-07-14_11-03-41

rsignell-usgs commented 5 years ago

If I save the notebook as a gist, the hover works correctly: 2019-07-14_11-10-47

rsignell-usgs commented 5 years ago

I discovered that downgrading from bokeh=1.2 to bokeh=1.1 did NOT fix the problem.

But downgrading to bokeh=1.0 DID fix the problem, but conda also downgraded most of the holoviz stack:

The following packages will be DOWNGRADED:

  bokeh                                        1.1.0-py37_0 --> 1.0.4-py37_1000
  holoviews                                     1.12.3-py_2 --> 1.11.0-py_0
  hvplot                                         0.4.0-py_1 --> 0.3.0-py_0
  panel                                    0.6.0-h5ca1d4c_0 --> 0.4.0-1
rsignell-usgs commented 5 years ago

@philippjfr , @jbednar, @jlstevens and @jsignell, any chance of getting this fixed before my HoloViz demo here at the ESIP Summer Meeting (Wednesday, July 17, 3:45pm PDT)?

I don't want to have to downgrade (and potentially break things) unless I have to. Thanks!

jsignell commented 5 years ago

Note that the change happened in holoviews between 1.12.1 and 1.12.2. I am moving the issue to that repo.

Screen Shot 2019-07-15 at 4 29 40 PM
rsignell-usgs commented 5 years ago

I can confirm that pinning holoviews=1.12.1 in my environment.yml works! Thanks for the workaround, @jsignell.

rsignell-usgs commented 5 years ago

This is fixed in geoviews master.

github-actions[bot] commented 1 week ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.