feat: add radar pipeline #892
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: scepter914 <[email protected]>
…owarefoundation#4993) Signed-off-by: scepter914 <[email protected]>
…ion#4992) Signed-off-by: scepter914 <[email protected]>
Signed-off-by: scepter914 <[email protected]>
Signed-off-by: scepter914 <[email protected]>
…usion and fix launch order (autowarefoundation#5166) * fix(tier4_perception_launch): add parameters for light weight radar fusion and fix launch order Signed-off-by: scepter914 <[email protected]> * style(pre-commit): autofix * add far_object_merger_sync_queue_size param for package arg Signed-off-by: scepter914 <[email protected]> --------- Signed-off-by: scepter914 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
scepter914
requested review from
miursh,
YoshiRi and
yukkysaito
as code owners
7 tasks
Signed-off-by: scepter914 <[email protected]>
scepter914
force-pushed
the
feat/radar_pipeline
branch
from
356a4ec to
8d14fdf
Compare
Signed-off-by: scepter914 <[email protected]>
badai-nguyen
reviewed
Oct 2, 2023
Comment on lines
+108
to
+149
| <!-- roi based clustering --> | ||
| <group> | ||
| <include file="$(find-pkg-share image_projection_based_fusion)/launch/roi_pointcloud_fusion.launch.xml" if="$(var use_roi_based_cluster)"> | ||
| <arg name="input/camera_info0" value="$(var camera_info0)"/> | ||
| <arg name="input/rois0" value="$(var detection_rois0)"/> | ||
| <arg name="input/camera_info1" value="$(var camera_info1)"/> | ||
| <arg name="input/rois1" value="$(var detection_rois1)"/> | ||
| <arg name="input/camera_info2" value="$(var camera_info2)"/> | ||
| <arg name="input/rois2" value="$(var detection_rois2)"/> | ||
| <arg name="input/camera_info3" value="$(var camera_info3)"/> | ||
| <arg name="input/rois3" value="$(var detection_rois3)"/> | ||
| <arg name="input/camera_info4" value="$(var camera_info4)"/> | ||
| <arg name="input/rois4" value="$(var detection_rois4)"/> | ||
| <arg name="input/camera_info5" value="$(var camera_info5)"/> | ||
| <arg name="input/rois5" value="$(var detection_rois5)"/> | ||
| <arg name="input/camera_info6" value="$(var camera_info6)"/> | ||
| <arg name="input/rois6" value="$(var detection_rois6)"/> | ||
| <arg name="input/camera_info7" value="$(var camera_info7)"/> | ||
| <arg name="input/rois7" value="$(var detection_rois7)"/> | ||
| <arg name="input/rois_number" value="$(var image_number)"/> | ||
| <arg name="input/image0" value="$(var image_raw0)"/> | ||
| <arg name="input/image1" value="$(var image_raw1)"/> | ||
| <arg name="input/image2" value="$(var image_raw2)"/> | ||
| <arg name="input/image3" value="$(var image_raw3)"/> | ||
| <arg name="input/image4" value="$(var image_raw4)"/> | ||
| <arg name="input/image5" value="$(var image_raw5)"/> | ||
| <arg name="input/image6" value="$(var image_raw6)"/> | ||
| <arg name="input/image7" value="$(var image_raw7)"/> | ||
| <arg name="input/pointcloud" value="/perception/object_recognition/detection/pointcloud_map_filtered/pointcloud"/> | ||
| <arg name="output_clusters" value="roi_cluster/clusters"/> | ||
| </include> | ||
| </group> | ||
|
|
||
| <!-- simple_cluster_merger --> | ||
| <group> | ||
| <include file="$(find-pkg-share cluster_merger)/launch/cluster_merger.launch.xml" if="$(var use_roi_based_cluster)"> | ||
| <arg name="input/cluster0" value="euclidean_cluster/clusters"/> | ||
| <arg name="input/cluster1" value="roi_cluster/clusters"/> | ||
| <arg name="output/clusters" value="clusters"/> | ||
| </include> | ||
| </group> | ||
|
|
There was a problem hiding this comment.
@scepter914 I think this code is duplicated? It is already existed in.
I am sorry maybe it was my fault to put it after eculidean clustering.
There was a problem hiding this comment.
|
@scepter914 I was my mistake in that make roi_cluster_fusion disconnected with shape_estimation
Could you change back it to: |
Signed-off-by: scepter914 <[email protected]>
Signed-off-by: scepter914 <[email protected]>
Signed-off-by: scepter914 <[email protected]>
9 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Add radar pipeline.
Related links
Tests performed
Tests by rosbag
Notes for reviewers
Interface changes
Effects on system behavior
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.