Added a TODO comment about changes which should be done in GpuMatcher::match
authorLeonidBeynenson <leonid.beynenson@itseez.com>
Tue, 15 Jul 2014 09:26:32 +0000 (13:26 +0400)
committerLeonidBeynenson <leonid.beynenson@itseez.com>
Tue, 15 Jul 2014 09:27:59 +0000 (13:27 +0400)
modules/stitching/src/matchers.cpp

index a0cb132..e0e4779 100644 (file)
@@ -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;