Skip to content

[HOTFIX][ZEPPELIN-1656] z.show in Python interpreter does not work - #1628

Closed
agoodm wants to merge 2 commits into
apache:masterfrom
agoodm:patch-1
Closed

[HOTFIX][ZEPPELIN-1656] z.show in Python interpreter does not work#1628
agoodm wants to merge 2 commits into
apache:masterfrom
agoodm:patch-1

Conversation

@agoodm

Copy link
Copy Markdown
Member

What is this PR for?

There have been reports of #1534 causing the python interpreter to always show an error because z is not being set. As it turns out this is a result of improperly handling the case when matplotlib isn't found when initializing the interpreter.

What type of PR is it?

Bug Fix

What is the Jira issue?

ZEPPELIN-1656

How should this be tested?

Run any simple python paragraph.

Questions:

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

@agoodm

Copy link
Copy Markdown
MemberAuthor

@bzz please test ASAP!

@bzz

bzz commented Nov 13, 2016

Copy link
Copy Markdown
Member

Will test right now and post back

@bzz

bzz commented Nov 14, 2016

Copy link
Copy Markdown
Member

Tested with and without matplotlib - both cases work fine!

Also

%python
a = 'a'
print "{}".format(a)

works as expected as well!

👍 thank you for prompt fix, looks awesome to me, will merge to master as hotfix asap, if there is no further discussion

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

@agoodm@bzz