feat(pointcloud_preprocessor): support for 3d distortion correction algorithm and refactor distortion correction node #7137

vividf · 2024-05-27T08:14:28Z

Description

This PR solved the issue #6657.
This PR provides an option for the user to choose a 3d distortion corrector instead of using the current 2d distortion corrector.

Note that by using this, the time will increase around 50 percent.
This PR also refactored the distortion correction node to make the code cleaner and support different undistortion strategies (ex: hybrid 2d/3d) in the future.

Changes:

once the node gets the TF information between imu, baselink, and lidar, it will stop getting the transformation again.
add 3d undistorted algorithm
refactor the code.
Add parameter file, parameter schema, and launch for this node.

Tests performed

Run the node

ros2 launch pointcloud_preprocessor distortion_corrector.launch.xml

I tested this PR and ensured the pointcloud output is the same and the processing time has a negligible time difference (the result is different every time).

Before the PR
Input: mirror pointcloud (58 frames)
distortion corrector: 2d

	Minimum	Maximum	Average
Time	7.15	15.29	10.09

After the PR
Test 1:

	Minimum	Maximum	Average
Time	7.516	12.719	9.47

Test2:

	Minimum	Maximum	Average
Time	7.5	14.9	10.5

Input: mirror pointcloud (58 frames)
distortion corrector: 3d

After the PR

	Minimum	Maximum	Average
Time	13.02	22.26	16.05

Per point calculation time

test with 58 frames * 3 times
2d algorithm

Before PR

	Minimum	Maximum	Average	Std Dev	50%	99%
Time	4.000000e-08	3.256100e-05	5.722510e-08	3.861742e-08	5.600000e-08	7.000000e-08

After PR

	Minimum	Maximum	Average	Std Dev	50%	99%
Time	4.400000e-08	3.476100e-05	5.898946e-08	4.029356e-08	5.800000e-08	7.200000e-08

Pointcloud comparison

@meliketanrikulu
Checked distortion corrector input point cloud and output point cloud for understanding difference coming with this PR. I did this test by viewing the moments when it passed through speed bumps.
Before this PR:

Blue pc : Input of the distortion corrector
White pc : Output of the distortion corrector
We can see here pc which is output of the distortion corrector does not changes in direction z

After this PR:

Blue pc : Input of the distortion corrector
White pc : Output of the distortion corrector

We can see here pc which is output of the distortion corrector is changes in direction z.
After seeing this change, I tested it with ground segmentation to see if it provided an improvement.

Test with Ground Segmentation:

You can see below that ground segmentation adds ground points to non-ground points when passing through speed bump before the changes occur.
Before 3d distortion corrector (ground segmentation test) Video link is here

I observed that this error did not occur after checkout the 3d distortion corrector branch.
After 3d distortion corrector (ground segmentation test) Video link is here

Notes for reviewers

Interface changes

ROS Topic Changes

ROS Parameter Changes

Effects on system behavior

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

I've confirmed the contribution guidelines.
The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

The PR follows the pull request guidelines.
The PR has been properly tested.
The PR has been reviewed by the code owners.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

There are no open discussions or they are tracked via tickets.
The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: vividf <[email protected]>

…tion_correction

Signed-off-by: vividf <[email protected]>

…ctor_node

Signed-off-by: vividf <[email protected]>

sensing/pointcloud_preprocessor/CMakeLists.txt

Signed-off-by: vividf <[email protected]>

knzo25 · 2024-06-04T09:04:53Z

@vividf
This PR and #7031 contain all the core changes to the distortion correction and thus makes no sense (to me) to have them split like this. If you only want us to check the refactor part in this PR, should the base not be the other PR rather than main? Otherwise it is really double work.

knzo25 · 2024-06-04T09:06:08Z

(I am aware that you want to merge the other one first, in which case this PR automatically would show only the rafactor changes, but 1) main is blocked right now. 2) the PRs should be merged rather together)

Signed-off-by: vividf <[email protected]>

meliketanrikulu

Hello @vividf . I tested again with your new updates. I get same results. There doesn't seem to be any problem. Thanks for your your work. LGTM.

Related Tests

Distortion Corrector Input/Output point cloud Difference on speed bumps:

Green: Input of 3D distortion corrector
Pink: Output of the 3D distortion corrector
The output of the Distortion Corrector approximates the location of the objects on the map. (When I visualized the map on this image, I did not add it because it looked complicated, but I observed that it approached the location on the map).
I tested with ground segmentation.
The problem caused by wobbling of the point cloud while passing through the speed bump caused ground points to be added to non-ground points. I observed that it was corrected with 3D distortion corrector.
Test video is here
White point cloud is non-ground points in the video

knzo25 · 2024-07-01T11:05:28Z

@vividf
In case you have not done this before, I will leave the comment here:
It seems some of your commits were not signed off, which causes DCO to fail (it is required).
If you see the instructions, you can rebase to fix it, which is the clean way of doing things, but if you prefer, you can also click the Set DCO to pass button (please do not forget the -s from now on)

…ctor_node

vividf · 2024-07-02T06:46:34Z

@drwnz This PR requires your approval 😃

...d_preprocessor/include/pointcloud_preprocessor/distortion_corrector/distortion_corrector.hpp

Signed-off-by: vividf <[email protected]>

…f/autoware.universe into refactor/distortion_corrector_node

drwnz

LGTM 👍

drwnz · 2024-07-03T05:48:40Z

@vividf it would be good to sign all your commits to pass DCO.

Signed-off-by: vividf <[email protected]>

…_corrector_node

…lgorithm and refactor distortion correction node (autowarefoundation#7137) * add support for 3d distortion correction Signed-off-by: vividf <[email protected]> * change parameter back to default and do small refactor Signed-off-by: vividf <[email protected]> * init version, need to double check Signed-off-by: vividf <[email protected]> * fix the logic error Signed-off-by: vividf <[email protected]> * temporary save, need to delete some code * init done, need to check for 3d as time is high Signed-off-by: vividf <[email protected]> * fix error Signed-off-by: vividf <[email protected]> * temporaily save * clean code Signed-off-by: vividf <[email protected]> * remove unused parameters Signed-off-by: vividf <[email protected]> * fix constructor error Signed-off-by: vividf <[email protected]> * fix spell errors Signed-off-by: vividf <[email protected]> * fix more spell errors Signed-off-by: vividf <[email protected]> * add undistorter to library Signed-off-by: vividf <[email protected]> * fix: fix cmake and change class name Signed-off-by: vividf <[email protected]> * Update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix imu to IMU Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: remove old naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * fix: fix file name and variable name Signed-off-by: vividf <[email protected]> * chore: fix invlaid virtual function definitions Signed-off-by: vividf <[email protected]> * fix: add sophus in package dependency Signed-off-by: vividf <[email protected]> * chore: remove brackets Signed-off-by: vividf <[email protected]> * chore: fix algorithm Signed-off-by: vividf <[email protected]> * chore: remove timestamp_field_name and add default parameter for 3d distortion Signed-off-by: vividf <[email protected]> * chore: fix known limits explanation Signed-off-by: vividf <[email protected]> * feat: add parameter schema and launch file for distortion correction node Signed-off-by: vividf <[email protected]> * chore: fix function name Signed-off-by: vividf <[email protected]> * chore: fix IMU function name Signed-off-by: vividf <[email protected]> * chore: fix twist and imu iterator function name Signed-off-by: vividf <[email protected]> * fix: add inline for undistortPointcloud function Signed-off-by: vividf <[email protected]> * chore: move varialbe to const Signed-off-by: vividf <[email protected]> * chore: fix grammar error Signed-off-by: vividf <[email protected]> * fix: fix inline function Signed-off-by: vividf <[email protected]> * chore: solve conflicts Signed-off-by: vividf <[email protected]> * fix: fix bug in previous code Signed-off-by: vividf <[email protected]> * fix: fix the template naming Signed-off-by: vividf <[email protected]> * fix: fix the component test Signed-off-by: vividf <[email protected]> * fix: solve conflicts Signed-off-by: vividf <[email protected]> --------- Signed-off-by: vividf <[email protected]> Co-authored-by: David Wong <[email protected]> Signed-off-by: palas21 <[email protected]>

…lgorithm and refactor distortion correction node (autowarefoundation#7137) * add support for 3d distortion correction Signed-off-by: vividf <[email protected]> * change parameter back to default and do small refactor Signed-off-by: vividf <[email protected]> * init version, need to double check Signed-off-by: vividf <[email protected]> * fix the logic error Signed-off-by: vividf <[email protected]> * temporary save, need to delete some code * init done, need to check for 3d as time is high Signed-off-by: vividf <[email protected]> * fix error Signed-off-by: vividf <[email protected]> * temporaily save * clean code Signed-off-by: vividf <[email protected]> * remove unused parameters Signed-off-by: vividf <[email protected]> * fix constructor error Signed-off-by: vividf <[email protected]> * fix spell errors Signed-off-by: vividf <[email protected]> * fix more spell errors Signed-off-by: vividf <[email protected]> * add undistorter to library Signed-off-by: vividf <[email protected]> * fix: fix cmake and change class name Signed-off-by: vividf <[email protected]> * Update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix imu to IMU Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: remove old naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * fix: fix file name and variable name Signed-off-by: vividf <[email protected]> * chore: fix invlaid virtual function definitions Signed-off-by: vividf <[email protected]> * fix: add sophus in package dependency Signed-off-by: vividf <[email protected]> * chore: remove brackets Signed-off-by: vividf <[email protected]> * chore: fix algorithm Signed-off-by: vividf <[email protected]> * chore: remove timestamp_field_name and add default parameter for 3d distortion Signed-off-by: vividf <[email protected]> * chore: fix known limits explanation Signed-off-by: vividf <[email protected]> * feat: add parameter schema and launch file for distortion correction node Signed-off-by: vividf <[email protected]> * chore: fix function name Signed-off-by: vividf <[email protected]> * chore: fix IMU function name Signed-off-by: vividf <[email protected]> * chore: fix twist and imu iterator function name Signed-off-by: vividf <[email protected]> * fix: add inline for undistortPointcloud function Signed-off-by: vividf <[email protected]> * chore: move varialbe to const Signed-off-by: vividf <[email protected]> * chore: fix grammar error Signed-off-by: vividf <[email protected]> * fix: fix inline function Signed-off-by: vividf <[email protected]> * chore: solve conflicts Signed-off-by: vividf <[email protected]> * fix: fix bug in previous code Signed-off-by: vividf <[email protected]> * fix: fix the template naming Signed-off-by: vividf <[email protected]> * fix: fix the component test Signed-off-by: vividf <[email protected]> * fix: solve conflicts Signed-off-by: vividf <[email protected]> --------- Signed-off-by: vividf <[email protected]> Co-authored-by: David Wong <[email protected]>

…lgorithm and refactor distortion correction node (#7137) * add support for 3d distortion correction Signed-off-by: vividf <[email protected]> * change parameter back to default and do small refactor Signed-off-by: vividf <[email protected]> * init version, need to double check Signed-off-by: vividf <[email protected]> * fix the logic error Signed-off-by: vividf <[email protected]> * temporary save, need to delete some code * init done, need to check for 3d as time is high Signed-off-by: vividf <[email protected]> * fix error Signed-off-by: vividf <[email protected]> * temporaily save * clean code Signed-off-by: vividf <[email protected]> * remove unused parameters Signed-off-by: vividf <[email protected]> * fix constructor error Signed-off-by: vividf <[email protected]> * fix spell errors Signed-off-by: vividf <[email protected]> * fix more spell errors Signed-off-by: vividf <[email protected]> * add undistorter to library Signed-off-by: vividf <[email protected]> * fix: fix cmake and change class name Signed-off-by: vividf <[email protected]> * Update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix imu to IMU Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: remove old naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * fix: fix file name and variable name Signed-off-by: vividf <[email protected]> * chore: fix invlaid virtual function definitions Signed-off-by: vividf <[email protected]> * fix: add sophus in package dependency Signed-off-by: vividf <[email protected]> * chore: remove brackets Signed-off-by: vividf <[email protected]> * chore: fix algorithm Signed-off-by: vividf <[email protected]> * chore: remove timestamp_field_name and add default parameter for 3d distortion Signed-off-by: vividf <[email protected]> * chore: fix known limits explanation Signed-off-by: vividf <[email protected]> * feat: add parameter schema and launch file for distortion correction node Signed-off-by: vividf <[email protected]> * chore: fix function name Signed-off-by: vividf <[email protected]> * chore: fix IMU function name Signed-off-by: vividf <[email protected]> * chore: fix twist and imu iterator function name Signed-off-by: vividf <[email protected]> * fix: add inline for undistortPointcloud function Signed-off-by: vividf <[email protected]> * chore: move varialbe to const Signed-off-by: vividf <[email protected]> * chore: fix grammar error Signed-off-by: vividf <[email protected]> * fix: fix inline function Signed-off-by: vividf <[email protected]> * chore: solve conflicts Signed-off-by: vividf <[email protected]> * fix: fix bug in previous code Signed-off-by: vividf <[email protected]> * fix: fix the template naming Signed-off-by: vividf <[email protected]> * fix: fix the component test Signed-off-by: vividf <[email protected]> * fix: solve conflicts Signed-off-by: vividf <[email protected]> --------- Signed-off-by: vividf <[email protected]> Co-authored-by: David Wong <[email protected]>

…lgorithm and refactor distortion correction node (autowarefoundation#7137) * add support for 3d distortion correction Signed-off-by: vividf <[email protected]> * change parameter back to default and do small refactor Signed-off-by: vividf <[email protected]> * init version, need to double check Signed-off-by: vividf <[email protected]> * fix the logic error Signed-off-by: vividf <[email protected]> * temporary save, need to delete some code * init done, need to check for 3d as time is high Signed-off-by: vividf <[email protected]> * fix error Signed-off-by: vividf <[email protected]> * temporaily save * clean code Signed-off-by: vividf <[email protected]> * remove unused parameters Signed-off-by: vividf <[email protected]> * fix constructor error Signed-off-by: vividf <[email protected]> * fix spell errors Signed-off-by: vividf <[email protected]> * fix more spell errors Signed-off-by: vividf <[email protected]> * add undistorter to library Signed-off-by: vividf <[email protected]> * fix: fix cmake and change class name Signed-off-by: vividf <[email protected]> * Update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: update sensing/pointcloud_preprocessor/docs/distortion-corrector.md Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix imu to IMU Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: fix company name Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: remove old naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * chore: change boolean variable naming Co-authored-by: David Wong <[email protected]> Signed-off-by: vividf <[email protected]> * fix: fix file name and variable name Signed-off-by: vividf <[email protected]> * chore: fix invlaid virtual function definitions Signed-off-by: vividf <[email protected]> * fix: add sophus in package dependency Signed-off-by: vividf <[email protected]> * chore: remove brackets Signed-off-by: vividf <[email protected]> * chore: fix algorithm Signed-off-by: vividf <[email protected]> * chore: remove timestamp_field_name and add default parameter for 3d distortion Signed-off-by: vividf <[email protected]> * chore: fix known limits explanation Signed-off-by: vividf <[email protected]> * feat: add parameter schema and launch file for distortion correction node Signed-off-by: vividf <[email protected]> * chore: fix function name Signed-off-by: vividf <[email protected]> * chore: fix IMU function name Signed-off-by: vividf <[email protected]> * chore: fix twist and imu iterator function name Signed-off-by: vividf <[email protected]> * fix: add inline for undistortPointcloud function Signed-off-by: vividf <[email protected]> * chore: move varialbe to const Signed-off-by: vividf <[email protected]> * chore: fix grammar error Signed-off-by: vividf <[email protected]> * fix: fix inline function Signed-off-by: vividf <[email protected]> * chore: solve conflicts Signed-off-by: vividf <[email protected]> * fix: fix bug in previous code Signed-off-by: vividf <[email protected]> * fix: fix the template naming Signed-off-by: vividf <[email protected]> * fix: fix the component test Signed-off-by: vividf <[email protected]> * fix: solve conflicts Signed-off-by: vividf <[email protected]> --------- Signed-off-by: vividf <[email protected]> Co-authored-by: David Wong <[email protected]>

vividf and others added 13 commits May 16, 2024 12:21

add support for 3d distortion correction

5f85981

Signed-off-by: vividf <[email protected]>

Merge branch 'autowarefoundation:main' into feature/support_3d_distro…

f30364f

…tion_correction

change parameter back to default and do small refactor

026edf3

Signed-off-by: vividf <[email protected]>

init version, need to double check

89216a1

Signed-off-by: vividf <[email protected]>

fix the logic error

da8f829

Signed-off-by: vividf <[email protected]>

temporary save, need to delete some code

80d88e1

init done, need to check for 3d as time is high

4d96e94

Signed-off-by: vividf <[email protected]>

fix error

b990188

Signed-off-by: vividf <[email protected]>

temporaily save

23e4ab9

clean code

3830fe6

Signed-off-by: vividf <[email protected]>

remove unused parameters

bf07078

Signed-off-by: vividf <[email protected]>

fix constructor error

93fa49f

Signed-off-by: vividf <[email protected]>

Merge branch 'autowarefoundation:main' into refactor/distortion_corre…

4b7f9e8

…ctor_node

github-actions bot added type:documentation Creating or refining documentation. (auto-assigned) component:sensing Data acquisition from sensors, drivers, preprocessing. (auto-assigned) labels May 27, 2024

vividf mentioned this pull request May 27, 2024

feat(pointcloud_preprocessor): support 3d distortion corrector for distortion corrector node #7031

Closed

7 tasks

vividf added 2 commits May 27, 2024 17:34

fix spell errors

4023409

Signed-off-by: vividf <[email protected]>

fix more spell errors

eb420a2

Signed-off-by: vividf <[email protected]>

vividf marked this pull request as ready for review May 27, 2024 15:07

vividf requested review from amc-nu, badai-nguyen, knzo25, kyoichi-sugahara, miursh, YoshiRi and yukkysaito as code owners May 27, 2024 15:07

yukkysaito reviewed May 27, 2024

View reviewed changes

sensing/pointcloud_preprocessor/CMakeLists.txt Outdated Show resolved Hide resolved

add undistorter to library

8e8dfb9

Signed-off-by: vividf <[email protected]>

vividf and others added 2 commits June 27, 2024 18:17

Merge branch 'main' into refactor/distortion_corrector_node

e969356

fix: fix bug in previous code

9fef37b

Signed-off-by: vividf <[email protected]>

meliketanrikulu reviewed Jun 28, 2024

View reviewed changes

Merge branch 'autowarefoundation:main' into refactor/distortion_corre…

386a70c

…ctor_node

meliketanrikulu mentioned this pull request Jul 2, 2024

Improve Roll and Pitch estimation in Localization #6993

Closed

vividf mentioned this pull request Jul 2, 2024

feat(pointcloud_preprocessor): add unit test for distortion corrector node #7783

Closed

Merge branch 'main' into refactor/distortion_corrector_node

2ab7edd

drwnz requested changes Jul 3, 2024

View reviewed changes

...d_preprocessor/include/pointcloud_preprocessor/distortion_corrector/distortion_corrector.hpp Outdated Show resolved Hide resolved

vividf added 2 commits July 3, 2024 14:40

fix: fix the template naming

3c3ca35

Signed-off-by: vividf <[email protected]>

Merge branch 'refactor/distortion_corrector_node' of github.com:vivid…

c395fab

…f/autoware.universe into refactor/distortion_corrector_node

vividf requested a review from drwnz July 3, 2024 05:42

drwnz approved these changes Jul 3, 2024

View reviewed changes

Merge branch 'main' into refactor/distortion_corrector_node

a6db67b

vividf enabled auto-merge (squash) July 3, 2024 06:25

vividf disabled auto-merge July 3, 2024 06:45

vividf added 3 commits July 3, 2024 15:54

fix: fix the component test

d9c8b65

Signed-off-by: vividf <[email protected]>

fix: solve conflicts

7f76efb

Signed-off-by: vividf <[email protected]>

Merge remote-tracking branch 'upstream/main' into refactor/distortion…

ee8ace5

…_corrector_node

vividf merged commit a7b8a10 into autowarefoundation:main Jul 3, 2024
28 of 30 checks passed

meliketanrikulu mentioned this pull request Jul 3, 2024

The distortion corrector package doesn't compensate for all motion distortion #6657

Closed

5 tasks

vividf mentioned this pull request Jul 5, 2024

ground remover package cannot delete ghost points appears front of car when passing speedbump #3673

Closed

3 tasks

meliketanrikulu mentioned this pull request Jul 5, 2024

fix(ekf_localizer): add imu option for roll pitch and height estimation #7875

Closed

vividf mentioned this pull request Jul 12, 2024

distortion_corrector cannot compensate point cloud while passing over speed bump #4961

Closed

3 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(pointcloud_preprocessor): support for 3d distortion correction algorithm and refactor distortion correction node #7137

feat(pointcloud_preprocessor): support for 3d distortion correction algorithm and refactor distortion correction node #7137

vividf commented May 27, 2024 •

edited

Loading

knzo25 commented Jun 4, 2024

knzo25 commented Jun 4, 2024

meliketanrikulu left a comment

knzo25 commented Jul 1, 2024

vividf commented Jul 2, 2024

drwnz left a comment

drwnz commented Jul 3, 2024

feat(pointcloud_preprocessor): support for 3d distortion correction algorithm and refactor distortion correction node #7137

feat(pointcloud_preprocessor): support for 3d distortion correction algorithm and refactor distortion correction node #7137

Conversation

vividf commented May 27, 2024 • edited Loading

Description

Related links

Tests performed

Pointcloud comparison

Test with Ground Segmentation:

Notes for reviewers

Interface changes

ROS Topic Changes

ROS Parameter Changes

Effects on system behavior

Pre-review checklist for the PR author

In-review checklist for the PR reviewers

Post-review checklist for the PR author

knzo25 commented Jun 4, 2024

knzo25 commented Jun 4, 2024

meliketanrikulu left a comment

Choose a reason for hiding this comment

Related Tests

knzo25 commented Jul 1, 2024

vividf commented Jul 2, 2024

drwnz left a comment

Choose a reason for hiding this comment

drwnz commented Jul 3, 2024

vividf commented May 27, 2024 •

edited

Loading