From 8578f9c5651308bd5ae265e061e902fff46e58da Mon Sep 17 00:00:00 2001 From: LeonidBeynenson Date: Tue, 15 Jul 2014 13:26:32 +0400 Subject: [PATCH] Added a TODO comment about changes which should be done in GpuMatcher::match --- modules/stitching/src/matchers.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/modules/stitching/src/matchers.cpp b/modules/stitching/src/matchers.cpp index a0cb13203..e0e477940 100644 --- a/modules/stitching/src/matchers.cpp +++ b/modules/stitching/src/matchers.cpp @@ -212,6 +212,10 @@ void GpuMatcher::match(const ImageFeatures &features1, const ImageFeatures &feat descriptors1_.upload(features1.descriptors); descriptors2_.upload(features2.descriptors); + //TODO: NORM_L1 allows to avoid matcher crashes for ORB features, but is not absolutely correct for them. + // The best choice for ORB features is NORM_HAMMING, but it is incorrect for SURF features. + // More accurate fix in this place should be done in the future -- the type of the norm + // should be either a parameter of this method, or a field of the class. BFMatcher_GPU matcher(NORM_L1); MatchesSet matches;