xrviz Version on binder does not plot.

Currently, the version that is opened with the binder does not make the plot, even if I click the Plot button.

Apr 23 '20 09:04 koldunovn

I believe this is the same as https://github.com/intake/xrviz/pull/69 I will try to make some time to put things right.

Apr 23 '20 12:04 martindurant

Works OK now?

Apr 23 '20 14:04 martindurant

I clicked the binder link, ran the notebook, got a plot, but then (yipes) I get this: https://www.youtube.com/watch?v=50dylR5d3-M

Apr 23 '20 17:04 rsignell-usgs

Constant unzooming! Is there an issue for this?

Apr 23 '20 17:04 martindurant

Do you think it's a holoviews/geoviews issue or an xrviz issue?

Apr 23 '20 18:04 rsignell-usgs

As far as I know, xrviz creates the plot and then leaves it alone. It is plausible that the layout is being recalculated somehow... @philippjfr , your thoughts?

Apr 23 '20 19:04 martindurant

Yeah something in holoviews. It's probably to do with aspect handling. Is this using recent HoloViews yet?

Apr 23 '20 19:04 philippjfr

The script uses "current" versions (whatever conda gives us from conda-forge on py37)

Apr 23 '20 19:04 martindurant

@rsignell-usgs , if you still have this open, would be good to post the versions binder ended up using

Apr 23 '20 19:04 martindurant

# packages in environment at /srv/conda/envs/notebook:
#
# Name                    Version                   Build  Channel

bokeh                     2.0.1            py37hc8dfbb8_0    conda-forge
datashader                0.10.0                     py_0    conda-forge
holoviews                 1.13.2             pyh9f0ad1d_0    conda-forge
hvplot                    0.5.2                      py_0    conda-forge
geoviews                  1.8.1                      py_0    conda-forge
geoviews-core             1.8.1                      py_0    conda-forge
panel                     0.9.5                      py_1    conda-forge
python                    3.7.6           h8356626_5_cpython    conda-forge

Apr 23 '20 20:04 rsignell-usgs

Okay, so definitely latest, I'll try to reproduce.

Apr 23 '20 20:04 philippjfr

Okay, I see the same thing. Will see if I can get a 1.13.3 release with a fix out very soon.

Apr 23 '20 21:04 philippjfr

The plot works for me now on the binder, thanks!

Apr 24 '20 08:04 koldunovn

@philippjfr I'm so glad you could replicate the zooming out feedback loop issue and it sounds like you see the solution also. Awesome!

Apr 24 '20 13:04 rsignell-usgs

Just wish we had an automated way to catch these kinds of regressions. Unfortunately testing client-server interactions in a browser is hard.

Apr 24 '20 13:04 philippjfr