Skip to content
Snippets Groups Projects

Major undistortion refactoring

Merged Nicolas Cadart requested to merge RefactorUndistortion into master
  1. Feb 08, 2021
    • Nicolas Cadart's avatar
      [fix] Add undistortion support in pose graph optimization · 6c1ad2ea
      Nicolas Cadart authored
      After PGO, frames were only rigidly aggregated.
      Now, it is possible to undistort them using optimized motion.
      This also fixes a small time duration display as timers should only be reset on a hard reset.
      6c1ad2ea
    • Nicolas Cadart's avatar
      [feat][refactor] Split APPROXIMATED undistortion mode into ONCE and REFINED · 6694ce06
      Nicolas Cadart authored
      ONCE: undistortion is done ony once at the beginning of localization step.
      REFINED: ONCE + undistortion refined at the end of each ICP-LM iteration.
      
      Current*Keypoints become CurrentRaw*Keypoints, raw extracted keypoints in BASE without undistortion.
      Current*Keypoints are the keypoints clouds in BASE, with undistortion (if enabled).
      
      Remove check in AddFrames verifying if we deal with first frame or not.
      We need to run Localization step to update Current*Keypoints from CurrentRaw*Keypoints.
      EgoMotion and Localization steps are safe if previous pointclouds are empty.
      6694ce06
  2. Feb 04, 2021
  3. Feb 02, 2021
    • Nicolas Cadart's avatar
      [deprec] Remove OPTIMIZED undistortion mode · d5d2f1cc
      Nicolas Cadart authored
      This mode is rarely used as it lead to much slower and unstabler optimization.
      Tests have shown that the results with this mode are always worse than with the APPROXIMATED mode.
      State of the art papers always use simple undistortion such as the APPROXIMATED mode.
      d5d2f1cc
Loading