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

Add Incremental Matching #1091

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

luca-morreale
Copy link

Hello,
I add the possibility to compute the matches incrementally.
It starts from a set of matches, otherwise it just computes all of them, and from them it computes the missing ones.
A match is computed only if at least one of the two views involved is not present in any of the pre-computed match.

- Add function to filter out pairs of views when both views are already present in the computed matches
- Edit force flag to add incremental matching
- Add incremental matching, it computes matches only between views that are not both already present in computed matches
Fix variable type from size_t to const IndexT to make it compliant with the convention adopted
@pmoulon
Copy link
Member

pmoulon commented Oct 9, 2017

Thx.

Seems like it does not compiles on Unix:
https://travis-ci.org/openMVG/openMVG/jobs/285531810#L1921

@luca-morreale
Copy link
Author

You are right, I actually forgot to commit a change.
Now it should compile properly.

@pmoulon
Copy link
Member

pmoulon commented Oct 9, 2017

No worries, the continuous integration is here to check that (safe compilation and unit test launching).

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

Successfully merging this pull request may close these issues.

None yet

2 participants