-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Ensure plotlyjs loaded #466
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
Conversation
@@ -171,25 +177,13 @@ def iplot(figure_or_data, show_link=True, link_text='Export to plot.ly', | |||
|
|||
Example: | |||
``` | |||
from plotly.offline import init_notebook_mode, iplot | |||
init_notebook_mode() | |||
from plotly.offline import, iplot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
random comma
'</script>' | ||
'').format(script=get_plotlyjs()) | ||
|
||
display(HTML(script_inject)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐄 could use less escaping by following original formatting, but not needed!
Looks good to me! @theengineear Do you have time to look over this PR? |
|
||
@raises(plotly.exceptions.PlotlyError) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
Looks great! Thanks for doing that! 💃 |
What release of plotly will have this issue? This is blocking our production use of plotly. |
Thanks! I saw you pushed 👍 |
Fixes issues with offline mode not working after refreshes.
In Brief:
init_notebook_mode