Skip to content

[refact] Add a velodyne reader and stream sources

Timothée Couble requested to merge refact/useVelodyneSpecializeReaderProxy into master

Goal

Refact the interpreters and the calibration.

Changes

  • Import default calibration files at build time
  • Add reader that combines LiDAR + Pose information
  • Add specific Velodyne reader and stream sources: The new interpreter architecture replace the common LidarReader with a specialized one (for each interpreter) This allow us to customize more what interpreter can do, and also helps with default values.
  • Add a lidar_interpreters proxy xml for Velodyne, this class will be used to help LidarView know which proxy to create in the configuration dialog.

Minor changes

  • Fix: Change type of DualReturnFilter to int
  • Use condition on select a VLS-128 for the option use old firing timing
  • Correct PPSSynct field out of SourceProxy
  • Use an enum class for SensorType
  • Fix Velodyne tests
    • Calibration files were renamed with a _, had to rename also data files
    • vtkLidarPoseReader takes both interpreters (LiDAR + pose), with a new output for pose

MR Dependences

Depends on lidarview-core!474 (merged), lidarview-core!475 (merged)

Edited by Julia Sanchez

Merge request reports