-
Notifications
You must be signed in to change notification settings - Fork 285
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
Allow input on comm messages #285
Comments
Probably makes sense. |
Any update to this issue? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The execute request message has an
allow_stdin
parameter to instruct the kernel if input could be handled. However, the comm messages do not have a similar parameter, which means, for example, you can't set a breakpoint in a comm message handler and have interactive debugging.If we had such a parameter in comm messages, they would no longer be symmetric, but this is still an important usecase.
See jupyter-widgets/ipywidgets#1603, CC @TimShawver.
For an implementation, it looks like essentially, we'd need to do something like https://github.com/ipython/ipykernel/blob/8acaee80c83c9a64a22afde522f9df2d0c3638e4/ipykernel/ipkernel.py#L200 and https://github.com/ipython/ipykernel/blob/8acaee80c83c9a64a22afde522f9df2d0c3638e4/ipykernel/ipkernel.py#L206 when processing comm messages, i.e.:
Thoughts?
The text was updated successfully, but these errors were encountered: