search

intake / xrviz

Interactive visualisation interface for Xarrays
https://xrviz.readthedocs.io
BSD 3-Clause "New" or "Revised" License
105 stars 21 forks source link

Revert "unpin holoviews" #46

Closed rsignell-usgs closed 4 years ago

rsignell-usgs commented 4 years ago

Reverts intake/xrviz#45

rsignell-usgs commented 4 years ago

@jhamman I merged because yes, you no longer need to pin holoviews, but then realized you need to do something like:

  - pyviz/label/dev::hvplot
  - pyviz/label/dev::holoviews
  - pyviz/label/dev::panel
  - pyviz/label/dev::datashader
  - metpy
  - numpy
  - xarray
  - pip
  - pip:
    - git+https://github.com/pyviz/geoviews.git

to get the tooltips to accurately show lon/lat.

jhamman commented 4 years ago

Ugh! This is too bad. xrviz is basically unusable in its current state. A package this small shouldn't be pinning a widely used dependency like this. My preference would be forgo tool tips today and push holoviews to make a release ASAP. Thoughts?

rsignell-usgs commented 4 years ago

@jhamman, indeed, this is a sad state of affairs. Hopefully things will change : https://github.com/pyviz/hvplot/issues/324

My preference would be to use the dev channels and geoviews master for now:

 channels:
  - conda-forge
dependencies:
  - cartopy
  - fsspec
  - pyviz/label/dev::hvplot
  - pyviz/label/dev::holoviews
  - pyviz/label/dev::panel
  - pyviz/label/dev::datashader
  - metpy
  - numpy
  - xarray
  - pip
  - pip:
    - git+https://github.com/pyviz/geoviews.git

and then switch to conda-forge when the next release is made, after which @philippjfr says things should calm down for a while.

philippjfr commented 4 years ago

Will try my best to release datashader, holoviews, geoviews and panel today. @jsignell probably has a better handle on how far out an hvPlot release is.

jhamman commented 4 years ago

Thanks @philippjfr - those efforts will be much appreciated.