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

[bin] meshroom_batch: support multiple init nodes #2137

Merged
merged 4 commits into from
Aug 4, 2023

Conversation

mugulmd
Copy link
Contributor

@mugulmd mugulmd commented Aug 1, 2023

Description

Modify meshroom_batch command line to support pipelines with multiple init nodes, such as the cameraTracking pipeline.

The --input and --inputRecursive arguments syntax used to be input1 input2 input3 ... (each input being either an image, a folder or a SfMData file), now it is NodeAName:input1;input2;input3 NodeBName:input4.

For example with a cameraTracking pipeline where the shot to track is in a folder named /path/to/my/shot and the distortion grid is just one image named distogrid.exr the command line would be:

meshroom_batch -i CameraInit_1:/path/to/my/shot CameraInit_2:distogrid.exr -p cameraTracking

When the pipeline used has only one init node (most common use case) there is no need to specify the NodeName: part:

meshroom_batch -i /some/images;/more/images -p photogrammetry

Note

The --scale argument has been removed since it was redundant.

@mugulmd mugulmd added the feature new feature (proposed as PR or issue planned by dev) label Aug 1, 2023
@mugulmd mugulmd added this to the Meshroom 2023.3.0 milestone Aug 1, 2023
@mugulmd mugulmd self-assigned this Aug 1, 2023
@mugulmd mugulmd marked this pull request as ready for review August 1, 2023 10:57
@cbentejac cbentejac merged commit 68e9474 into develop Aug 4, 2023
2 checks passed
@cbentejac cbentejac deleted the mug/batchMultiInit branch August 4, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature new feature (proposed as PR or issue planned by dev)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants