-
Notifications
You must be signed in to change notification settings - Fork 198
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use is_complete request to indicate complete input #20
Conversation
kc = self.kernel_client | ||
if kc is None: | ||
return | ||
from jupyter_client import BlockingKernelClient |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be a regular import rather than a local one? jupyter_client is a plain dependency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
regular, this was just lazy hacking as I was figuring it out.
Thanks for getting round to this. |
Thanks for reviewing - I had this mostly put together when we last talked about it a few weeks ago, but forgot about it. Thanks for reviewing. I think I did things a few different ways, and there are likely a couple of redundant bits in here, still. I'll give it another pass tomorrow. |
1623e61
to
6851c89
Compare
Okay, review should be addressed. Some redundant bits from earlier attempts have been removed. indent from is_complete_reply is used now, instead of hardcoded value. |
@@ -57,6 +57,7 @@ def gui_excepthook(exctype, value, tb): | |||
) | |||
|
|||
from jupyter_core.application import JupyterApp | |||
from jupyter_client import BlockingKernelClient |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This import is unnecessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, removed.
instead of IPython InputSplitter
use is_complete request to indicate complete input
instead of IPython InputSplitter.
Adds a BlockingKernelClient to perform the is_complete_request, since the current use of
_is_complete
can't easily be made async.based on #19