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

[sfm] New sfmFilter software: split sfm based on filenames regex #1769

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jmelou
Copy link
Member

@jmelou jmelou commented Oct 25, 2024

SfMFilter can be used to create a new sfmdata containing a subset of views, selected using a regex on their name.
SfMTransfert can now select view using their poseID (used in MV-PS)

Create new sfmdatas by selecting views from sfmData with regex.
@fabiencastan fabiencastan changed the title [sfm] Add Sfm Filter [sfm] New sfmFilter software: split sfm based on filenames regex Oct 29, 2024
@fabiencastan fabiencastan added this to the 3.3.0 milestone Oct 29, 2024

CmdLine cmdline("AliceVision sfmFilter");
cmdline.add(requiredParams);
//cmdline.add(optionalParams);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused line, to be removed.

"Path to the output SfMData file.\n")
("outputSfMData_unselected,o", po::value<std::string>(&outputSfMFilenameUnselected)->required(),
"Path to the output SfMData file.\n");

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// clang-format on

clang-format is disabled at the beginning of the parameters listing but it is never re-enabled. It should be re-enabled as soon as the parameters have been described.

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.

4 participants