Hi Alexander, thank you for contributing Matplotlib integration improvements!
You are right, it's odd and this is not your text editor's fault - our current Python interpreter implementation is quite early stages and these are artefacts of the way it currently works. Sometimes it does not accept valid Python files [1]. I'm working on improving it right now [2], by changing the underneath mechanism of communication with Python process, which should address this issues, as well as few others [3], like windows support, better error detection, etc. I hope to submit a PR next week, meanwhile the best way to debug your changes is to enable logging [4] by adding a line to log4j.properties " log4j.logger.org.apache.zeppelin.python.PythonInterpreter=DEBUG" (i.e by creating it in ./python/src/test/resources/ if you want it to work only in unit-tests, or just changing code to `log.warn` to always log it) in order to be able to see what exactly actual Python repl process is not happy about. Hope this helps! 1. https://issues.apache.org/jira/browse/ZEPPELIN-1133 2. https://issues.apache.org/jira/browse/ZEPPELIN-1325 3. https://issues.apache.org/jira/browse/ZEPPELIN-1133?jql= project%20%3D%20ZEPPELIN%20AND%20component%20%3D%20python-interpreter%20AND% 20resolution%20%3D%20Unresolved 4. https://github.com/apache/zeppelin/blob/master/python/src/main/java/org/ apache/zeppelin/python/PythonInterpreter.java#L221 -- Alex On Sat, Aug 13, 2016, 01:04 Goodman, Alexander (398K) <alexander.goodman@jpl .nasa.gov> wrote: > An update: It seems like the problem might lie in my editor. After a bit > of hacking into the log fiIes, I found that there were several > IndentationErrors being reported, starting at the PyZeppelinContext > constructor, resulting in a seemingly empty class once bootstrap.py is > processed. It's still odd because again I can import bootstrap.py in my > local python installation and use z.show() without IndentationErrors. > > On Fri, Aug 12, 2016 at 7:19 AM, Goodman, Alexander (398K) < > alexander.good...@jpl.nasa.gov> wrote: > >> Hi, >> >> I am attempting to improve the matplotlib plotting capabilities in the >> python interpreter (see the ZEPPELIN-1318 JIRA issue) and have ran into >> some issues. Specifically, after saving my changes to bootstrap.py and >> rebuilding, I get the following error message after running the unit tests: >> >> AttributeError: 'PyZeppelinContext' object has no attribute 'show' >> expected:<SUCCESS> but was:<ERROR> >> >> PythonInterpreterPandasSqlTest.sqlOverTestDataPrintsTable:129 Traceback >> (most recent call last): >> >> File "<stdin>", line 1, in <module> >> >> The same error message also appeared when I attempted to run the simple >> matplotlib example notebook. This seems to happen if I make any changes to >> the show_matplotlib() method in bootstrap.py, including one line changes >> that don't actually do anything. I also tested the module for syntax errors >> in my local python installation and everything seems to check out, so I >> can't understand where the problem lies. I am fairly certain my changes are >> sound (see [1]). I tested this for a zeppelin build without any additional >> options (mvn clean package -DskipTests) both with the modified and >> unmodified source code, the latter having no issues. >> >> [1] - https://github.com/agoodm/zeppelin/commit/10c06a9ceb7be940 >> 85ea39ebaad8beab7e0ff024 >> >> Any help would be appreciated. >> >> Thanks, >> Alex >> >> -- >> Alex Goodman >> Data Scientist I >> Science Data Modeling and Computing (398K) >> Jet Propulsion Laboratory >> California Institute of Technology >> Tel: +1-818-354-6012 >> > > > > -- > Alex Goodman > Data Scientist I > Science Data Modeling and Computing (398K) > Jet Propulsion Laboratory > California Institute of Technology > Tel: +1-818-354-6012 >
