Skip to content

added iplot_mpl() and plotly_takeover() #366

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

Closed
wants to merge 1 commit into from

Conversation

arsenovic
Copy link
Contributor

iplot_mpl() allows offline matplotlib figure conversion
for ipython notebook inline mode.

plotly_takeover() is a compound command that allows an entire
notebook session that was written using matplotlib to use offline
plotly without any modification.

the later idea was taken from
https://github.com/jakevdp/mpld3/blob/202792bcddbfe957ade08a5085691ab176a4563d/mpld3/_display.py#L363

`iplot_mpl()` allows offline matplotlib figure conversion
for ipython notebook inline mode.

`plotly_takeover()`  is a compound command that allows an entire
notebook session that was written using matplotlib to use offline
plotly without any modification.

the later idea was taken from
https://github.com/jakevdp/mpld3/blob/202792bcddbfe957ade08a5085691ab176a4563d/mpld3/_display.py#L363
@@ -171,6 +184,34 @@ def iplot(figure_or_data, show_link=True, link_text='Export to plot.ly',
height=height, width=width)))


def iplot_mpl(mpl_fig,mpl_to_plotly_kw={},iplot_kw={}):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add spacing between the args
def iplot_mpl(mpl_fig, mpl_to_plotly_kw={}, iplot_kw={}):

@cldougl
Copy link
Member

cldougl commented Jan 10, 2016

Thanks @arsenovic !
Can you add a super simple use example to the docstring? I'll add some tests.

@arsenovic
Copy link
Contributor Author

i probably wont make any of these changes, but feel free to edit as you wish.

@cldougl cldougl mentioned this pull request Jan 15, 2016
@cldougl
Copy link
Member

cldougl commented Jan 15, 2016

moved: #386

@cldougl cldougl closed this Jan 15, 2016
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

Successfully merging this pull request may close these issues.

2 participants