Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/2580
---
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/2580
[ZEPPELIN-2924] Fix broken javascript source map
### What is this PR for?
Fixed broken javascript source map. The coverage loader should be used only
for test env.
**that's rea
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2558
@ruseel Thanks for the update. Could you resolve zjffdu's last comment?
---
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2580
[ZEPPELIN-2924] Fix broken javascript source map
### What is this PR for?
Fixed broken javascript source map. The coverage loader should be used only
for test env.
**that's reall
Hoon Park created ZEPPELIN-2924:
---
Summary: javascript source map is broken
Key: ZEPPELIN-2924
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2924
Project: Zeppelin
Issue Type: Bug
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2578
LGTM.
---
Hoon Park created ZEPPELIN-2923:
---
Summary: ipython related message is displayed when enabling a
conda env
Key: ZEPPELIN-2923
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2923
Project: Zeppelin
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2578
@zjffdu I created one. Thanks.
- https://issues.apache.org/jira/browse/ZEPPELIN-2923
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2578
@1ambda This is due to IPythonInterpreter, could create a ticket ? I will
fix it in another PR.
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2578
Checked that `conda list` works beautifully. But I am getting this ipython
related message when activate a new env. Can we remove this message which is
not related to conda at all.
![imag
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2579
@malayhm Hi, Thanks for the contribution.
I just read the JIRA issue and have a question. Could you elaborate the
situation when
> sometimes user want to delete the last empty par
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2578
fixed test case and conda list feature
---
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2578
@1ambda okay :)
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2578
Tested locally and works well. @cloverhearts Can we fix `conda list`
command as well? it will need exactly the same code with `conda install`.
LGTM except for it.
---
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2578
@zjffdu Sure, Thank you :)
---
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2569
Merge if no more discussion.
---
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2572
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2578
Thanks @cloverhearts , Is it possible to add unit test for
`PythonCondaInterpreter` ? Regarding the CI failure, I know this failure and
will fix it in another PR
---
GitHub user malayhm opened a pull request:
https://github.com/apache/zeppelin/pull/2579
Zeppelin-2922 Allow removing last paragrap
### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2578
The problem of CI is irrelevant to this pr.
```
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 12.462 sec
<<< FAILURE! - in
org.apache.zeppelin.interpreter.remot
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/2575
---
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/2574
---
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2574
I have separated this PR.
I'm sorry for the confusion.
https://github.com/apache/zeppelin/pull/2578
---
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2578
@1ambda @zjffdu @felixcheung
Sorry.
I divided the existing PR by the minimum function.
Can you review this PR again?
Thank you for your help :)
---
GitHub user cloverhearts opened a pull request:
https://github.com/apache/zeppelin/pull/2578
[ZEPPELIN-2921] does not work conda environment in python interpreter
### What is this PR for?
It seems that the environment of the python interpreter has changed to the
py4j environmen
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/2574
Thank you @1ambda @zjffdu @felixcheung
I think my code with various purposes is distracting you.
And, unfortunately, my English ability is difficult to describe in a
sophisticated way
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2574
@cloverhearts There is no problem about python lib paths.
`%python.conda install` and `%python conda list` commands are the problem.
They don't aware activated env by Zeppelin.
So
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2574
@cloverhearts
`There is only one python library path to be added.`
Do you mean the py4j ? IIUC, PythonInterpreter will add py4j to PYTHONPATH,
it doesn't relate with `PythonCondaInterpreter
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2574
@cloverhearts There is no problem about python lib paths.
`%python.conda install` command is the problem.
So you don't need to write code for python path.
