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

add magic for matplotlib display(pyspark or pyspark3) #487

Merged

Conversation

wangqiaoshi
Copy link

now sparkmagic must be pass data to local then matplot.
it is expensive if the data is big

@Tagar
Copy link

Tagar commented Jan 18, 2019

This is a great improvement .

A screenshot of an example usage would be nice.. or sparkmagic documentation updated.

Thanks!

@wangqiaoshi
Copy link
Author

This is a great improvement .

A screenshot of an example usage would be nice.. or sparkmagic documentation updated.

Thanks!

ok,i add screenshot.

@Tagar
Copy link

Tagar commented Feb 11, 2019

Thanks @wangqiaoshi

@apetresc could you please help review and hopefully merge this?

It seems to be a great improvement for larger visualizations - so viz can happen on Spark driver and then only visualization itself will have to travel over network to Jupyter.

@brookwc
Copy link

brookwc commented Apr 30, 2019

@Tagar @apetresc hey! This diff is quite useful to us. Has it been merged into trunk?

@Tagar
Copy link

Tagar commented Jun 25, 2019

@itamarst can this be part of the newer release you're preparing ?
thank you

@itamarst itamarst changed the base branch from master to server-side-image-rendering July 12, 2019 18:21
@itamarst
Copy link
Contributor

Merging to branch so I can iterate on this.

@itamarst itamarst merged commit 5873b55 into jupyter-incubator:server-side-image-rendering Jul 12, 2019
@itamarst itamarst mentioned this pull request Jul 12, 2019
4 tasks
@itamarst
Copy link
Contributor

Hi, could you all try the version in #552? Adds support for usage in normal notebooks, not just the PySpark kernel.

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.

4 participants