-
-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Using with Hydrogen? #42
Comments
Glad you like the package! There are some issues with loading up external javascript into atom. I think @MikeInnes might know the most about this, so I'll ping him here and hope he can offer some more expertise. I think once we solve this problem we will have this in both Hydrogen and Juno, which will be great. |
Yeah I can probably save you some time here. One easy way to get around this is to walk the result HTML and evaluate the content of script tags via My best (untested) idea right now is to load Plotlyjs via a |
Is it possible to just visualize the plots in a Blink/Electron window outside of Atom, the way PlotlyJS does with command-line Julia? I like using Hydrogen for interacting with code, but I don’t mind doing my data visualization independent of Atom, e.g., in a pop-out window. The in-Atom non-interactive PNG files that PyPlot produces aren’t really worth much for me. |
This worked for me.
I think the tricky part is that PlotlyJS.jl tries to detect if jupyter is being used and if so it will default to trying to display plots in the notebook. In this case hydrogen does use the Jupyter kernel (Julia), but not from within a notebook. We need a way in IJulia.jl to determine if Jupyter is being run from in a notebook or elsewhere and then we can default to electron windows if we aren't in the notebook. |
Is there any known way to use this with Hydrogen? Plots appears as zero-size divs at the moment although the plotly.js code is successfully injected into the div:
One error does appear in Atom's js console (details here):
I'll start looking into this, but opening this issue in case someone else has already figured this out.
Also: Discovering this package has been the highlight of my week. Thanks!
The text was updated successfully, but these errors were encountered: