merge with IJuliaWidgets? about interact.jl HOT 6 CLOSED

Interact doesn't require IJulia/IPython. Eventually I want to use Interact to have these widgets ported to LightTable. This arrangement is good for specifying dependencies for each of these (e.g. IJuliaWidgets requires IJulia 0.1.13-.

from interact.jl.

What does IJuliaWidgets even do? I'm just doing using Interact in IJulia and @manipulate works fine.

from interact.jl.

Ah, that's because Interact checks to see if you are including it from IJulia and if so loads IJuliaWidgets. It will throw a warning if IJuliaWidgets is not installed. All the IJulia-specific stuff (e.g. Comm handling) and hacks go in IJuliaWidgets while Interact strives for a neat API both for the user and developers of other IDEs.

from interact.jl.

@shashi, in that case, the REQUIRE file of Interact.jl needs to list IJuliaWidgets anyway, as Interact won't work in IJulia if the user forgets to Pkg.add it. However, it is weird to have it as a separate package because, as far as I can tell, a user will never ever type using IJuliaWidgets explicitly: it is purely an internal module used secretly within Interact.

My suggestion would be to just put IJuliaWidgets in Interact/src/IJuliaWidgets, as a subdirectory. This will make it easier to keep track of the code, and but it will still be a (sub)module (Interact.IJuliaWidgets). This will also play nicely with JuliaLang/julia#6195 (you will just be able to declare module IJuliaWidgets requires IJulia and it will do what you want).

One of my motivations for this is that I'd like to see Interact and React moved to JuliaLang at some point soon, and it will be easier for us to keep track of if there are fewer modules. Another motivation is bug reports: when people file issues for Interact, some of the issues will actually apply to IJuliaWidgets (which people won't even be aware of because it is included silently), and it will be be annoying to constantly refer to issues from Interact from IJuliaWidgets commits.

from interact.jl.

(Note that if you make IJuliaWidgets an internal module within Interact that is only loaded when IJulia is running, then you don't need to REQUIRE IJulia in Interact, because Interact will still be loadable without IJulia.)

from interact.jl.

Okay for now I moved IJuliaWidgets to src/IJulia and include src/IJulia/setup.jl if Main.IJulia. JuliaLang/julia#6195 will be nice to have! This has annoyed me a lot too and I am happy for the change. :)

from interact.jl.