search

JuliaPluto / PlutoSliderServer.jl

Web server to run just the `@bind` parts of a Pluto.jl notebook
https://computationalthinking.mit.edu/
The Unlicense
130 stars 17 forks source link

Wrong bond connections graph behavior #117

Closed ctrekker closed 1 year ago

ctrekker commented 1 year ago

The following notebook will reproduce the issue: Screenshot from 2023-06-07 12-10-07

The value of f depends on both x and y, implying that x and y are codependent. It seems like PlutoSliderServer's analysis doesn't recognize this: Screenshot from 2023-06-07 12-13-31

I haven't done much more testing beyond finding this simplified MWE, but it seems this is an issue with codependents and friends, since Pluto calculates the topology of this notebook without any issues.

ctrekker commented 1 year ago

Strangely everything works just fine when I look at it with a debugger....

fonsp commented 1 year ago

super nice! can you add this to one of the test notebooks?

Op 7 jun. 2023 om 20:29 heeft Connor Burns @.***> het volgende geschreven:

 Strangely everything works just fine when I look at it with a debugger....

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.

ctrekker commented 1 year ago

This is only a bug on the precomputed sliders PR. In the latest commit it should be fixed there!

Pangoraw commented 1 year ago

This look likes https://github.com/fonsp/Pluto.jl/issues/2427 so maybe the bug is only on 1.9.

ctrekker commented 1 year ago

This look likes fonsp/Pluto.jl#2427 so maybe the bug is only on 1.9.

I bet that's it! That looks like pretty much an identical bug, and the issue was fixed after I updated Pluto from 0.19.9 to >=0.19.19 in the slider server project toml.