Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
I will merge it if no more comments
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
ping Leemoonsoo @1ambda @felixcheung @prabhjyotsingh for review
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
@Leemoonsoo I have fixed the issue, could you try it again ? Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Tried last commit.
I installed jupyter, IPython, ipykernel, grpcio in my conda environment and
i configured my python interpreter like
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
@Leemoonsoo @1ambda @felixcheung @prabhjyotsingh I made several follow up
commit. All the features in this stage is done. Please help review that. I
believe it would be a big improvement for
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Push another commit to support streaming output
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Push another commit to use IPython in PySparkInterpreter if IPython is
available , otherwise fallback to the old implementation.
---
If your project is set up for it, you can reply to this email
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Just push another commit to use IPythonInterpreter first, if it is not
available, I will fallback it to the old PythonInterpreter.
---
If your project is set up for it, you can reply to this
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
@1ambda I did some investtigation on that, and puts some links in
https://issues.apache.org/jira/browse/ZEPPELIN-2756, hope it would be helpful
for you.
---
If your project is set up for it, you
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Let create a PR to fix the ASCII issue, we can test this PR based on the
ASCII PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Thanks @Leemoonsoo for catching that. This is due to I miss cursor for code
completion. Just pushed another commit to fix that and add more unit test for
it.
```
// there's no
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Is it expected completion list?
---
If your project is set up for it, you
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Yes, that works. It is covered in unit test
(https://github.com/apache/zeppelin/pull/2474/files#diff-d9c8252d37a199d13b7157e88041c0e9R124).
Also I manually verified it
---
If your project is
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Tested 88a1f2f and ZeppelinContext works well.
Does auto completion supposed to work?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
@Leemoonsoo Regarding the ZeppelinContext, could you try that again ? It is
covered in unit test, so should be able to work. I force committed it
yesterday, the latest commit Id is
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
Thanks for testing @Leemoonsoo . These special characters are for font
color, create ZEPPELIN-2756 for that which might need help from @1ambda
---
If your project is set up for it, you can reply
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2474
@Leemoonsoo @1ambda @felixcheung @prabhjyotsingh Please help review.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
17 matches
Mail list logo
