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

Add note to documentation wrt VS Code launch.json template #14

Open
HarryDC opened this issue Nov 29, 2023 · 1 comment
Open

Add note to documentation wrt VS Code launch.json template #14

HarryDC opened this issue Nov 29, 2023 · 1 comment

Comments

@HarryDC
Copy link

HarryDC commented Nov 29, 2023

Currently the template for launch.json comes up with something similar to

{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python: Remote Attach",
            "type": "python",
            "request": "attach",
            "connect": {
                "host": "localhost",
                "port": 5678
            },
            "pathMappings": [
                {
                    "localRoot": "${workspaceFolder}",
                    "remoteRoot": "."
                }
            ],
            "justMyCode": true
        }
    ]
}

But with a development setup (i.e. slicer built from source) the pathMappings parameter here might the debugger from stepping into slicer python code, this might also be a problem how i set up the workspace though, but removing the pathMappings entry allowed stepping into the base slicer python

@lassoan
Copy link
Member

lassoan commented Nov 29, 2023

Yes, path mappings must be removed. We have already described this in the setup instructions, but it is practically impossible to make people read the documentation carefully.

Any suggestions for improving the documentation is welcome but probably we would need to somewhow change this setting automatically or find a way to make things work with the default path mapping.

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

No branches or pull requests

2 participants