-
Notifications
You must be signed in to change notification settings - Fork 294
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
Implement %pdb support in notebooks #6374
Comments
The Source: https://ipython.readthedocs.io/en/stable/interactive/magics.html#magic-pdb |
The |
@int19h Does Python extension support |
debugpy doesn't have an interactive command mode like pdb does. Also, I think this command is hardcoded in the notebooks to specifically invoke pdb? If it's actually a hook that can invoke an arbitrary function, then having it invoke However, this particular scenario (breaking on exception) could be handled simply by running under debugger with User-uncaught Exceptions set to true. |
This is actually supported, via quick input |
Do you have more details? %pdb is supposed to trap exceptions and show them in the debugger. At least I think that's how it works. |
When an exception occurs, I see a quick input show up which lets me enter pdb commands. In Jupyter lab, an input box appears below the cell for pdb commands. So it seems to work as well as Jupyter lab. I don't expect it to integrate with the vscode debugger or anything like that. |
It seems unlikely to me that dropping into the full vscode from However, it might make sense to open a new issue to discuss/investigate a "drop into vscode debugger from running notebook" feature more thoroughly if that doesn't currently exist (or maybe better document how to set it up if it does). |
Can you explain more about what you're looking for? Debugging notebooks does exist with the "Debug cell" command |
Ahh, ok – no that is perfect. For some reason it wasn't clear to me that Debug Cell used the existing notebook kernel. |
Assuming we have debugging working can %pdb also work? Not sure what this does at the moment, but user's have been asking for this.
The text was updated successfully, but these errors were encountered: