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

StructureFromMotion: Add new inputs parameters #1980

Merged
merged 2 commits into from
May 10, 2023

Conversation

cbentejac
Copy link
Contributor

@cbentejac cbentejac commented Apr 13, 2023

Description

Add two new input parameters to the StructureFromMotion node:

  • nbFirstUnstableCameras which sets the number of cameras for which a bundle adjustment is performed every time a camera is added;
  • maxImagesPerGroup which sets the maximum number of cameras that can be added before the bundle adjustment has to be performed again.

By default, the bundle adjustment will be performed every time one of the first 30 cameras is added, as computations are not too expensive while there are few cameras. Past that number, the bundle adjustment will only be performed when up to 30 cameras have been added.

The node's version and the templates using the StructureFromMotion node are updated accordingly.

Related AliceVision pull request: alicevision/AliceVision#1413

Add two new input parameters, `nbFirstUnstableCameras` which sets the
number of cameras for which a bundle adjustment is performed every time
a camera is added, and `maxImagesPerGroup` which sets the maximum number
of cameras that can be added before the bundle adjustment has to be
performed again.

By default, the bundle adjustment will be performed every time one of
the first 30 cameras is added, as computations are not too expensive
while there are few cameras. Past that number, the bundle adjustment will
only be performed when up to 30 cameras have been added.

The node's version is updated accordingly.
@cbentejac cbentejac changed the title [nodes] StructureFromMotion: Add new inputs parameters StructureFromMotion: Add new inputs parameters Apr 13, 2023
@fabiencastan fabiencastan added this to the Meshroom 2023.2.0 milestone Apr 18, 2023
@fabiencastan fabiencastan added the feature new feature (proposed as PR or issue planned by dev) label Apr 18, 2023
@fabiencastan fabiencastan merged commit c66b86b into develop May 10, 2023
@fabiencastan fabiencastan deleted the dev/sfmExposeParams branch May 10, 2023 11:36
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