search

holoviz-topics / EarthSim

Tools for working with and visualizing environmental simulations.
https://earthsim.holoviz.org
BSD 3-Clause "New" or "Revised" License
65 stars 21 forks source link

next_n argument doesn't work with parambokeh #200

Closed sdc50 closed 5 years ago

sdc50 commented 6 years ago

Specifying the next_n argument when calling parambokeh.Widgets has no effect.

parambokeh.Widgets(parameterized_obj, next_n=1)

I would expect this to run the next cell when a parameter widget is updated but nothing happens.

jlstevens commented 6 years ago

Agreed. This sounds like a bug.

jbednar commented 6 years ago

I noticed that yesterday as well, but hadn't gotten around to filing a bug report. Thanks for doing it for me!

kcpevey commented 6 years ago

yes! I ran across that a while back but I thought I just misunderstood what it was supposed to do. Always better to file a report I suppose.

philippjfr commented 6 years ago

My best guess is that parambokeh (or its future replacements) will never support the next_n option because the JS code that paramnb uses to make it work is exceptionally hacky.

jbednar commented 6 years ago

We should definitely make it not accept the option, then! :-) In any case I think the recent Panel work will make it very unlikely that next_n would really be the best choice in any scenario.

philippjfr commented 6 years ago

Agreed, another problem with it is that even in paramnb it doesn't work in JupyterLab.

jlstevens commented 6 years ago

I also vote against next_n support due to how hacky it is to get working...