intake / xrviz

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

unpin holoviews in requirements.txt? #36

Closed jhamman closed 4 years ago

jhamman commented 4 years ago

Would it be possible to unpin holoviews? We're finding that using xrviz is making downstream package management at this time?

https://github.com/intake/xrviz/blob/e39c38a9004e445f52d611724f582c5b39c0da46/requirements.txt#L2

cc @scottyhq

martindurant commented 4 years ago

@hdsingh , is there any reason to keep this pin?

hdsingh commented 4 years ago

This specific version was added about 2 weeks ago to get correct lon/lat values on the hover. The latest version of holoviews doesn't show correct values. Everything else works fine with the latest version.

hdsingh commented 4 years ago

https://github.com/pyviz/holoviews/issues/3891

jhamman commented 4 years ago

This needs to be reopened.

hdsingh commented 4 years ago

This has been resolved in #48.