Skip to content

[dev] Add TaskThread Qt debugging#3038

Merged
cbentejac merged 3 commits intodevelopfrom
dev/add_task_thread_debugging
Apr 9, 2026
Merged

[dev] Add TaskThread Qt debugging#3038
cbentejac merged 3 commits intodevelopfrom
dev/add_task_thread_debugging

Conversation

@Alxiice
Copy link
Copy Markdown
Contributor

@Alxiice Alxiice commented Mar 10, 2026

Description

This PR adds a way to debug TaskThread.
Before this PR we cannot add a breakpoint on a TaskThread.

Content of the PR

If a DEBUGGING env var is set to True we enable the debugging on the task thread

How to use

In VSCode this launch.json can be used

{
    // 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": "Meshroom",
            "type": "debugpy",
            "request": "launch",
            "program": "meshroom/ui",
            "console": "integratedTerminal",
            "env": {
                // "MESHROOM_NODES_PATH": "${workspaceFolder}/tests/nodes",
                // "MESHROOM_PLUGINS_PATH": "/path/to/plugins",
                "MESHROOM_OUTPUT_QML_WARNINGS": "1",
                "MESHROOM_QML_DEBUG": "0",
                "MESHROOM_QML_DEBUG_PARAMS": "port:7789",
                "DEBUGGING": "1",
            },
            "args": [
                "/path/to/scene.mg"
            ],
            "subProcess": true,
        }
  }

@Alxiice Alxiice force-pushed the dev/add_task_thread_debugging branch from 5df3f6f to 177aca0 Compare March 10, 2026 15:32
@Alxiice Alxiice self-assigned this Mar 10, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 10, 2026

Codecov Report

❌ Patch coverage is 40.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.38%. Comparing base (1628f7a) to head (c205bec).
⚠️ Report is 4 commits behind head on develop.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
meshroom/core/attribute.py 40.00% 3 Missing ⚠️

❌ Your patch status has failed because the patch coverage (40.00%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3038      +/-   ##
===========================================
- Coverage    83.40%   83.38%   -0.03%     
===========================================
  Files           73       73              
  Lines         9870     9874       +4     
===========================================
+ Hits          8232     8233       +1     
- Misses        1638     1641       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread meshroom/core/taskManager.py
@Alxiice Alxiice force-pushed the dev/add_task_thread_debugging branch from 177aca0 to 95c828a Compare March 16, 2026 13:21
@cbentejac cbentejac added this to the Meshroom 2026.1.0 milestone Apr 9, 2026
@Alxiice Alxiice force-pushed the dev/add_task_thread_debugging branch from a39d3a1 to 2ced656 Compare April 9, 2026 08:32
@Alxiice Alxiice force-pushed the dev/add_task_thread_debugging branch from 2ced656 to c205bec Compare April 9, 2026 09:09
@cbentejac cbentejac merged commit 9bed2ab into develop Apr 9, 2026
7 of 8 checks passed
@cbentejac cbentejac deleted the dev/add_task_thread_debugging branch April 9, 2026 09:17
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.

3 participants