Resolve "Implementation of Falko lib"

Closes #26 (closed)

src/loop_closure_falko.h

@@ -105,16 +105,16 @@ public:
 
   /** \brief Create and update the scene struct with keypoints and descriptors
    **/
-  sceneFalkoBSCPtr extractScene(LaserScan &scan, LaserScanParams &scanParams) {
-    auto newScene = std::make_shared<sceneFalko<D>>();
-    auto scanFALKO =
-      loopClosureFalko<D, Extr, M>::convert2LaserScanFALKO(scan, scanParams);
+  sceneFalkoBSCPtr extractScene(LaserScan &_scan, LaserScanParams &_scan_params) {
+    auto new_scene = std::make_shared<sceneFalko<D>>();
+    auto scan_falko =
+    convert2LaserScanFALKO(_scan, _scan_params);
     // Extract keypoints
-    extract((*scanFALKO), (newScene->keypointsList));
+    extract(*scan_falko, (new_scene->keypointsList));
     // Compute descriptors
-    extractor_.compute(*scanFALKO, newScene->keypointsList,
-                       newScene->descriptorsList);
-    return newScene;
+    extractor_.compute(*scan_falko, new_scene->keypointsList,
+                       new_scene->descriptorsList);
+    return new_scene;
   }
 
   /** \brief Convert scans from laserscanutils::LaserScan to
@@ -132,11 +132,11 @@ public:
   /** \brief Create and update a matchLoopClosure struct with the info that is
     *produced when matching two given scenes
    **/
-  matchLoopClosurePtr matchScene(sceneFalkoBSCPtr scene1,
-                                 sceneFalkoBSCPtr scene2) {
-    std::vector<std::pair<int, int>> assoNN;
+  matchLoopClosurePtr matchScene(sceneFalkoBSCPtr _scene1,
+                                 sceneFalkoBSCPtr _scene2) {
+    std::vector<std::pair<int, int>> asso_nn;
     int matching_number =
-      matcher_.match(scene1->keypointsList, scene2->keypointsList, assoNN);
+      matcher_.match(_scene1->keypointsList, _scene2->keypointsList, asso_nn);
     auto new_match = std::make_shared<matchLoopClosure<D>>();
     new_match->keypointsNumberMatch = matching_number;
     if (matching_number > keypoints_number_th) {
@@ -144,7 +144,7 @@ public:
     } else {
       new_match->match = false;
     }
-    new_match->sceneTuple = std::make_tuple(scene1, scene2);
+    new_match->sceneTuple = std::make_tuple(_scene1, _scene2);
     return new_match;
   }
 
@@ -152,14 +152,14 @@ public:
     *closures
    **/
   std::list<matchLoopClosurePtr>
-  findLoopClosure(std::list<sceneFalkoBSCPtr> &l_scenes,
-                  const sceneFalkoBSCPtr &new_scene) {
-    int number_ref_sc = l_scenes.size();
+  findLoopClosure(std::list<sceneFalkoBSCPtr> &_l_scenes,
+                  const sceneFalkoBSCPtr &_new_scene) {
+    int number_ref_sc = _l_scenes.size();
     std::list<matchLoopClosurePtr> matchings;
     for (int i = 0; i < number_ref_sc; i++) {
-      auto l_front = l_scenes.begin();
+      auto l_front = _l_scenes.begin();
       std::advance(l_front, i);
-      auto new_match = matchScene(*l_front, new_scene);
+      auto new_match = matchScene(*l_front, _new_scene);
       matchings.push_back(new_match);
     }
     return matchings;