Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • S Slam
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 4
    • Issues 4
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 15
    • Merge requests 15
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • KEU-ComputerVision
  • Slam
  • Merge requests
  • !71

Fix Ceres residuals conflict with LV's

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nicolas Cadart requested to merge fix/CeresCostFunctionsConflictWithLV into master Dec 14, 2020
  • Overview 6
  • Commits 2
  • Pipelines 1
  • Changes 3

Strangely, when the SLAM is compiled within LV, some of the cost functions used are LV's, and not SLAM's. It's even more tricky : the constructor of these residuals could be the one implemented in SLAM, but the operator() was the one defined in LV. Since !66 (merged), the cost functions are not compatible with LV's, so in these cases, SLAM completely failed.

To ensure that the correct residuals implementations are used, the namespace have been changed so that the symbols differ.

This also add the missing include into the CMakeList to properly export headers. Strangely, the project could already compile without declaring this include header.

This MR also cleans the CeresCostFunctions.h file, that originally comes from LidarView code base. Most of the residuals defined in this file are not used in the SLAM, but only in LV. This MR removes them for more clarity and less maintenance.

@nick.laurenson @julia.sanchez @lea.vauchier PTAL

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/CeresCostFunctionsConflictWithLV