Skip to content

Improving Matplotlib Integration with Zeppelin - #1425

Closed
IshmeetKaur wants to merge 15 commits into
apache:masterfrom
IshmeetKaur:master
Closed

Improving Matplotlib Integration with Zeppelin#1425
IshmeetKaur wants to merge 15 commits into
apache:masterfrom
IshmeetKaur:master

Conversation

@IshmeetKaur

Copy link
Copy Markdown

What is this PR for?

This PR improves the matplotlib integration with zeppelin.

What type of PR is it?

Improvement

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1428

How should this be tested?

show() functionality in the PyZeppelinContext class should work with matplotlib.

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@Leemoonsoo

Copy link
Copy Markdown
Member

Tried and working well.
Could you try trigger CI again and see if it goes to green?

@Leemoonsoo

Copy link
Copy Markdown
Member

Realized that there's a very similar contribution #1422 created few days before.

@Leemoonsoo

Copy link
Copy Markdown
Member

Thanks @IshmeetKaur. I think i'll try to merge #1422 first because #1422 arrived earlier.

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto 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

@IshmeetKaur@Leemoonsoo