-
Notifications
You must be signed in to change notification settings - Fork 302
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
Decouple DS debug adapter tracker for the python tracker #3102
Comments
@karthiknadig is this breaking multi process scenarios now? |
Yes.
|
Validate that multiproc debugging works for regular python AND we still support debugging in the interactive window. The rest we can do later. |
Validated. Did a quick check to see if the debug location moved to the correct location while debugging some cells with function calls. |
VSCode allows one DebugAdapterTracker per language. Currently DS registers an debug adapter tracker named
debugLocationTracker
. This tracker run is the non-DS scenarios and is currently designed to be a singleton instance.DebugAdapterTracker should be created for each debug session since there can be multiple debug sessions. The DS
debugLocationTracker
overwrites the debug session field in it every time a new session is created. This is creates issues when tracking multi process scenarios where there can be multiple debug sessions.We need a common tracker that can handle singleton instances as needed by DS and per session instances as needed by debug session telemetry
The text was updated successfully, but these errors were encountered: