Resolve "Emplace API inconsistent with ProcessorTrackerFeature/Landmark functions"

Closes #191 (closed)

src/processor/processor_tracker.cpp

@@ -157,17 +157,14 @@ void ProcessorTracker::process(CaptureBasePtr const _incoming_ptr)
         {
             processKnown();
 
-            // eventually add more features
-            if (last_ptr_->getFeatureList().size() < params_tracker_->min_features_for_keyframe)
+            if (voteForKeyFrame() && permittedKeyFrame())
             {
-                //WOLF_TRACE("Adding more features...");
+                // process
                 processNew(params_tracker_->max_new_features);
-            }
 
-            if (voteForKeyFrame() && permittedKeyFrame())
-            {
-                // We create a KF
+                //TODO abort KF if last_ptr_->getFeatureList().size() < params_tracker_->min_features_for_keyframe
 
+                // We create a KF
                 // set KF on last
                 last_ptr_->getFrame()->setState(getProblem()->getState(last_ptr_->getTimeStamp()));
                 last_ptr_->getFrame()->setKey();
@@ -176,9 +173,6 @@ void ProcessorTracker::process(CaptureBasePtr const _incoming_ptr)
                 FrameBasePtr frm = getProblem()->emplaceFrame(NON_ESTIMATED, incoming_ptr_->getTimeStamp());
                 incoming_ptr_->link(frm);
 
-                // process
-                processNew(params_tracker_->max_new_features);
-
                 // Set state to the keyframe
                 last_ptr_->getFrame()->setState(getProblem()->getState(last_ptr_->getTimeStamp()));