Skip to content
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

[DO NOT MERGE] Specify that the debug messages follow the debug adapter protocol #502

Open
wants to merge 180 commits into
base: main
Choose a base branch
from

Conversation

SylvainCorlay
Copy link
Member

No description provided.

takluyver and others added 30 commits October 9, 2017 15:23
MetaKernelFinder -> KernelFinder
Prototype new kernel discovery machinery
The old URL points to a "This page has moved"-page
Updated URL for Jupyter Kernels in other languages
- use IOLoop.current over IOLoop.instance
- drop removed `loop` arg from PeriodicCallback
- deprecate now-unused IOLoopKernelRestarter.loop
- interrupt_mode="signal" is the default and current behaviour
- With interrupt_mode="message", instead of a signal, a
  `interrupt_request` message on the control port will be sent
Additional to the actual signal, send a message on the control port
this should allow ipykernel's wheel-installed specs to specify `python3` or `python2` and prevent
python2 kernels from launching with sys.executable if the Python version is 3.
A simple lead in to the 'kernel nanny' work, this adds a command so you
can do:
jupyter kernel --kernel python
SylvainCorlay and others added 15 commits July 27, 2019 10:21
Updates to the kernel messaging spec
cocalc now properly works; also add a concrete example of an "astral plane unicode" identifier, so anybody can easily see what happens in the Jupyter client of their choice.
Document debug_[request|reply] and debug_event
This change tolerates the existence of templated entries where the
templated value corresponds to an existing environment variable.
In such cases, those templated values will be substituted prior to
the kernel's launch.
Add support for templated env entries
Prevent two kernels to have the same ports assigned in MultiKernelManager
Prevent two kernels to have the same ports
Only cache ports if the cache_ports flag is set to True
use session_id and message counter.
This gives a unique id but uses much less time
@SylvainCorlay
Copy link
Member Author

cc @jtpio @afshin

@SylvainCorlay
Copy link
Member Author

SylvainCorlay commented Dec 30, 2019

We should probably add a description of the dumpCell and debugInfo, which are not in the DAP (but follow the DAP base protocol).

I think this could be done in a separate PR.

@jtpio
Copy link
Member

jtpio commented Dec 30, 2019

dumpCell and debugInfo are indeed important to the debug protocol, but fine to add in a different PR 👍

docs/messaging.rst Outdated Show resolved Hide resolved
@SylvainCorlay
Copy link
Member Author

Please do not merge. I am opening a JEP.

@SylvainCorlay SylvainCorlay changed the title Specify that the debug messages follow the debug adapter protocol (JEP to come) Specify that the debug messages follow the debug adapter protocol Dec 30, 2019
@jasongrout jasongrout changed the title (JEP to come) Specify that the debug messages follow the debug adapter protocol [DO NOT MERGE] Specify that the debug messages follow the debug adapter protocol Dec 30, 2019
Copy link
Member

@jasongrout jasongrout left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes solely to flag not to merge at this point, as Sylvain asks.

@SylvainCorlay
Copy link
Member Author

Note: this needs a rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet