Skip to content
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

[discussion] combining with Plotly&PlotlyJS #61

Open
mmikhasenko opened this issue Sep 27, 2021 · 6 comments
Open

[discussion] combining with Plotly&PlotlyJS #61

mmikhasenko opened this issue Sep 27, 2021 · 6 comments

Comments

@mmikhasenko
Copy link

Thank you so much for the package. I just posted my plotly figure online, finally, after several days of struggling.
The posting functionality was very hard to find since I started from plotlyjs.

Is there a reason to keep the package separate from plotlyjs?
Interestingly the plotting script that I wrote with using PlotlyJS just worked once I replaced the input using Plotly

@nicolaskruchten
Copy link

Hi there!

Many folks are very concerned about data privacy and would be very concerned if the code that does the plotting and the code that does the uploading to the cloud were in the same package, so we keep them separate :)

@sglyon
Copy link
Collaborator

sglyon commented Sep 27, 2021

Thanks @nicolaskruchten -- that's right!

The Plotly.jl package (this one!) imports and reexports everything from PlotlyJS and then adds just the post functionality to interact with the chart studio servers.

There are no additional plot creating methods here -- all of that lives in PlotlyJS

@mmikhasenko
Copy link
Author

Now it is getting clear for me, thanks a lot for the clarification. Could it be added to the documentation?
From the outside, it looked to me that PlotlyJS is the newer version of Plotly and the latter is getting unmaintained (build:error)

@mmikhasenko
Copy link
Author

Also, concerning the documentation of the Plotly.jl. I did not get what the end_point is good for.
I did not need it to post a plot (sorry for the ignorance). A sentence of the use case in the doc would help

@mmikhasenko
Copy link
Author

One more edition to README.md.
I just figured out how to post Plots.jl plot:

using Plots
import Plotly: post, signin
#
plotlyjs()
p = plot(rand(10))
plt_o = Plots.plotlyjs_syncplot(p) # that is what I was looking for
signin("...","...")
remote_plot = post(plt_o)

If you post it somewhere, you save someone a lot of time

@mmikhasenko
Copy link
Author

Just created #63 . It would close this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants