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

[question] How to define the default parameters for nodes added for new groups during live reconstruction? #2548

Open
robot-army opened this issue Sep 25, 2024 · 3 comments

Comments

@robot-army
Copy link

When I'm running "live reconstruction", I would like to define the settings for eg. FeatureExtraction2, but I can't see where to do it. I'd like to untick Force CPU extraction, and use sift rather than dspsift, for example.
image
image

  • OS: Win10
  • Python version [e.g. 2.6]
  • Qt/PySide version [e.g. 5.15.2]
  • Meshroom version: 2023.3.0
@robot-army
Copy link
Author

So I've had a bit of a browse around the code and it seems that the nodes are built according to this bit:

https://github.com/alicevision/Meshroom/blob/9088f0b10695443dcab3362f94527673d7195dbd/meshroom/multiview.py#L208C1-L209C66

That seems to make a new FeatureExtraction node according to the defaults here:
https://github.com/alicevision/Meshroom/blob/9088f0b10695443dcab3362f94527673d7195dbd/meshroom/nodes/aliceVision/FeatureExtraction.py#L65C1-L65C31

I wonder if the right thing to do would be to have the node built by multiview.py take on the settings from the matching node types that already exist in the graph, rather than the defaults. Another way could be to make it easy to change those defaults. I haven't seen anywhere to do that yet but I'll keep digging.

@robot-army
Copy link
Author

I've also tried making a new pipeline with the default values here changed, and that doesn't do it.

@robot-army
Copy link
Author

Another thing that hasn't worked: Saving a graph with modified values as a template. Any new nodes that are added when new pics are taken seem to pick up the defaults of dspsift and "force CPU extraction". Need to compile from source to edit those defaults?

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

No branches or pull requests

1 participant