When opening Lidarview, the default action should be "EnableAdvancedArrays"
Then when the user click on the button, the enable advanced arrays are ON.
The buttons should now represent the action "Disable advanced arrays"
The Reaction update the icon and the tooltip according to that.

[fix] Automatically update the pipeline after loading a lidar state

See merge request !141

Add lidar camera toolbar

See merge request !139

This "Lidar Camera Toolbar" is almost the same as the paraview's camera toolbar.
In paraview the "set view Direction" buttons reset the center of rotation to the center of the data
For Lidar data the center of data is less important than the position of the lidar.
So in this "Lidar camera toolbar", the "set view direction" reset the center of rotation to the lidar position (0,0,0)

This will allow us to instantiate our own camera toolbar

Fix spreadsheet view

See merge request !140

The option is saved in the settings and reapply everytime the spreadsheet is opened
When we open Veloview for the first time this option is set to false
This avoid having an empty spreadsheet an misleading the user.

Columns visibility was restored only in the first opening of LidarView.
For example :
if you open the spreadsheet (using ctrl-t), hide a specific column,
       close the spreadsheet (using ctrl-t) and reopen it, the column wasn't hide.

This was due to the check on the current object (if it was different thant the previous).
Actually we want to update the column selection everytime the spreadsheet is displayed.
Even if it's the same object or the same frame.

Fix test stream and remove warnings

See merge request !137

The reader test all frames of a pcap (even the first and the last one by enabling the corresponding option)
So to generate the data, you also need to check "Chow first and last frame options"
In case there is no uncomplete last frame (for a truncated pcap for example) this option create an empty frame at the end of the pcap.

So when you test such a data in stream mode there is a segfault at the last split frame :
There is no point in the last frame,
so a simple call of "split frame" does not push back the current frame in the lidarPacketInterpreter frame buffer
because there is a check on the number of point.

The only way to avoid this segfault is by forcing the splitFrame.
So an empty frame can be push back to the lidarPacketInterpreter frame buffer

[Fix] fix windows compilation of lqLoadLidarStateReaction

See merge request !138

MelanieCarriere authored 4 years ago and MelanieCarriere committed 4 years ago

With MSVC : a double[size] can not be instantiate with a non constant size.

[feature] Save and load lidar state

See merge request !135

MelanieCarriere authored 4 years ago and MelanieCarriere committed 4 years ago

The lidar state represents all the properties of a lidar source and ones of its subproxy (interpreter, ...)
A lidar source can be a lidarStream or a LidarReader.
This commit allow to save all or a part of theses properties in json file.
You can the reload them on a pipeline which already contains a lidar source

[Test] Add a file to help generate groundtruth for new test data

See merge request !136

MelanieCarriere authored 4 years ago and MelanieCarriere committed 4 years ago

The file will not work as python file. You need to :
- Update it with your own paths
- Copy-paste it in a programmable source in lidarview

Add kitti label reader and dataset writer

See merge request !124

In particular: fix potential issue with eigen when passing a parameter by value

This is a temporary implementation, this writer should be implemented as
a writer instead of a filter.

Kitti 3d object detection labels are stored in camera reference
coordinates, providing the calibration files enables projecting them to
the lidar coordinates system

This reader reads Kitti 3d object detection labels

Up to now, the number of digits was hardcoded to 10, which didn't match
the object detection dataset format with 6 digits.

Revert "[CI] Enable pointing to a SHA from another remote (than the one by default)."

See merge request !134

This reverts commit ef5b4b8f.

In the current gitlab-ci setup, the same source directory can be reused.
This can lead the command `git remote add <name>` to fail, and so the ci
pipeline, in case the remote already exist.

[CI] Enable pointing to a SHA from another remote (than the one by default).

See merge request !126

This is usefull when we have a public and a private repository of the same submodule.
The public repository of the application will point by default to the public submodule.
But we need to be able to launch the CI from the private submodule.

Add plane parameters output to vtkRansacPlaneModel filter

See merge request !128

[ci] Specify job-level timeout for the superbuild

See merge request !130

Fix MLSPosesSmoothing

See merge request !131