Skip to content

[core] Load valid plugins even if they contain no node#2989

Merged
servantftransperfect merged 1 commit intodevelopfrom
fix/loadPluginWithoutNodes
Feb 5, 2026
Merged

[core] Load valid plugins even if they contain no node#2989
servantftransperfect merged 1 commit intodevelopfrom
fix/loadPluginWithoutNodes

Conversation

@cbentejac
Copy link
Copy Markdown
Contributor

Description

This PR allows to properly load plugins that do not contain any nodes. As long as there is a Python package in the meshroom folder of the plugin, even if it is empty, the plugin will be fully loaded. This can be useful for example when wanting to load a plugin that only provides templates.

With this commit, a plugin that does not contain any `NodePlugin` may
be loaded. Prior to it, if no node was found, the plugin was not added
to the list of plugins to load, even if it was valid. We can now load
plugins that for example only have specific templates.
@cbentejac cbentejac added this to the Meshroom 2026.1.0 milestone Feb 5, 2026
@cbentejac cbentejac self-assigned this Feb 5, 2026
@servantftransperfect servantftransperfect merged commit 35381b9 into develop Feb 5, 2026
3 checks passed
@servantftransperfect servantftransperfect deleted the fix/loadPluginWithoutNodes branch February 5, 2026 10:24
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.67%. Comparing base (a7e41ae) to head (83f5beb).
⚠️ Report is 2 commits behind head on develop.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2989   +/-   ##
========================================
  Coverage    81.67%   81.67%           
========================================
  Files           69       69           
  Lines         9144     9144           
========================================
  Hits          7468     7468           
  Misses        1676     1676           

☔ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants