-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/object detector with scene objects mamager #115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: humble-devel
Are you sure you want to change the base?
Changes from all commits
f4815fc
292a128
b385f29
184fd7e
2851853
a2d8423
bc62e9d
2122572
5b11569
d1f2c5e
a61dac2
b5b5109
d97282c
7415f55
e4f5e5a
01a0ec8
a11b44b
7fa7138
4b8bcc9
2be0866
bb1771a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,7 +3,7 @@ | |
| ros__parameters: | ||
| camera: | ||
| i_mx_id: 184430108157970F00 | ||
| i_pipeline_type: RGB # RGB, RGBD | ||
| i_pipeline_type: CamArray # RGB, RGBD, CamArray | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What does this pipeline change? Does it decrease camera's FPS relative to RGB pipeline? |
||
| i_nn_type: None # rgb, spatial | ||
| rgb: | ||
| # TODO(issue#113) Unify ROI configuration for all cameras | ||
|
|
@@ -14,19 +14,25 @@ | |
| i_height: 704 # 720 | ||
| i_width: 704 # 1280 | ||
| i_output_isp: false | ||
| i_publish_topic: true | ||
| # ------------- | ||
| i_enable_preview: false | ||
| # i_enable_passthrough: true | ||
| # i_keep_preview_aspect_ratio: true | ||
| # i_preview_size: 416 | ||
| # stereo: | ||
| # i_subpixel: true | ||
| # spatial_bb_node: | ||
| # ros__parameters: | ||
| # desqueeze: true | ||
| # nn: | ||
| # i_disable_resize: true | ||
| # i_enable_passthrough: true | ||
| stereo: | ||
| i_publish_topic: true | ||
| i_subpixel: true | ||
| right: | ||
| i_publish_topic: true | ||
| i_height: 720 | ||
| i_width: 1280 | ||
| left: | ||
| i_publish_topic: true | ||
| i_height: 720 | ||
| i_width: 1280 | ||
| depth: | ||
| i_publish_topic: true | ||
|
Comment on lines
+26
to
+35
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These lines were for debugging purposes. It would be great to save them for later (i vote for commenting them and creating an issue for incorporating them into a proper launch file later). |
||
| /cam_tool_front: | ||
| ros__parameters: | ||
| camera: | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -14,8 +14,8 @@ | |
| camera_model="$(arg scene_camera_model)" | ||
| base_frame="$(arg scene_camera_base_frame)" | ||
| cam_pos_x="0.33" | ||
| cam_pos_y="0.014" | ||
| cam_pos_z="1.166" | ||
| cam_pos_y="-0.016" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the reason for camera's position change? |
||
| cam_pos_z="1.160" | ||
|
Comment on lines
+17
to
+18
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we need to the camera position change? |
||
| cam_roll="0" | ||
| cam_pitch="${pi / 2}" | ||
| cam_yaw="${pi}" | ||
|
|
||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please update the |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| object_pose_detector: | ||
| ros__parameters: | ||
| aruco_size: 0.02 | ||
| image_topic: "/cam_scene/rgb/image_rect" | ||
| cam_info_topic: "/cam_scene/rgb/camera_info" | ||
| output_frame: "base_link" | ||
| cam_frame: "cam_scene_rgb_camera_optical_frame" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -36,6 +36,10 @@ def __init__(self): | |
| self.description_cfg_pkg_name = "aegis_description" | ||
| self.description_cfg_pkg = FindPackageShare(self.description_cfg_pkg_name) | ||
|
|
||
| # TODO if working then remove lines | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Address this TODO |
||
| # self.scene_objects_cfg_pkg_name = "scene_objects_manager" | ||
| # self.scene_objects_cfg_pkg = FindPackageShare(self.scene_objects_cfg_pkg_name) | ||
|
|
||
| self.kinematics_cfg = PathJoinSubstitution( | ||
| [self.moveit_cfg_pkg, "config", "move_group", "kinematics.yaml"] | ||
| ) | ||
|
|
@@ -52,6 +56,10 @@ def __init__(self): | |
| [self.moveit_cfg_pkg, "config", "scene_objects.yaml"] | ||
| ) | ||
|
|
||
| self.object_pose_detector_cfg = PathJoinSubstitution( | ||
| [self.moveit_cfg_pkg, "config", "object_pose_detector_config.yaml"] | ||
| ) | ||
|
|
||
| self.rviz_cfg = PathJoinSubstitution( | ||
| [self.moveit_cfg_pkg, "config", "moveit.rviz"] | ||
| ) | ||
|
|
@@ -267,10 +275,10 @@ def prepare_static_tf_node(base_link: str, child_link: str) -> Node: | |
| def prepare_scene_objects_manager_node(paths: AegisPathsCfg) -> Node: | ||
| return Node( | ||
| package="scene_objects_manager", | ||
| executable="scene_objects_manager", | ||
| name="scene_objects_manager", | ||
| executable="scene_objects_manager_node", | ||
| output="screen", | ||
| arguments=["--cfg", paths.scene_objects_cfg, "--frame", "world"], | ||
| parameters=[paths.object_pose_detector_cfg], | ||
| ) | ||
|
|
||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -186,7 +186,7 @@ def _republish_last_tf(self): | |
| if self.last_object_tf is None: | ||
| return | ||
| rvec, tvec = self.last_object_tf | ||
| tf_camera_name = "cam_scene_rgb_camera_optical_frame" | ||
| tf_camera_name = "cam_scene_rgb_camera_optical_frame_cal" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why we needed to change this tf? |
||
| self._publish_detected_object_tf( | ||
| rvec=rvec, | ||
| tvec=tvec, | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,23 +1,10 @@ | ||
| T_base2cam: | ||
| - [0.0, 1.0, 0.0, 0.339468918] | ||
| - [1.0, 0.0, 0.0, -0.029242202] | ||
| - [0.0, 0.0, -1.0, 1.170050432] | ||
| - [0.0, 1.0, 0.0, 0.329395049] | ||
| - [1.0, 0.0, 0.0, -0.020626899] | ||
| - [0.0, 0.0, -1.0, 1.159807969] | ||
| - [0.0, 0.0, 0.0, 1.0] | ||
| T_cam2base: | ||
| - [0.0, 1.0, 0.0, 0.0292422] | ||
| - [1.0, 0.0, 0.0, -0.33946892] | ||
| - [0.0, 1.0, 0.0, 0.020626899] | ||
| - [1.0, 0.0, 0.0, -0.329395049] | ||
| - [0.0, 0.0, -1.0, 1.159807969] | ||
| - [0.0, 0.0, 0.0, 1.0] | ||
| - [0.0, 0.0, 0.0, 1.0] | ||
|
|
||
| # TODO(issue#100) Calibration needs to be corrected. The visible matrices are calculated based on position error without taking camera rotation into account | ||
|
|
||
| # T_base2cam: | ||
| # - [0.019393463529861155, 0.9996413820847947, 0.018466206863277983, 0.3240708510322008] | ||
| # - [0.999024067443758, -0.018641790273284053, -0.04004243153875939, -0.007406087495627406] | ||
| # - [-0.0396838284499529, 0.0192247465265037, -0.9990273283952478, 1.1742893794290938] | ||
| # - [0.0, 0.0, 0.0, 1.0] | ||
| # T_cam2base: | ||
| # - [0.019393463529861016, 0.9990240674437583, -0.039683828449952906, 0.047714301707007856] | ||
| # - [0.9996413820847948, -0.0186417902732842, 0.019224746526503706, -0.34666811181735363] | ||
| # - [0.018466206863277983, -0.04004243153875939, -0.9990273283952479, 1.1668662643689283] | ||
| # - [0.0, 0.0, 0.0, 1.0] |
Uh oh!
There was an error while loading. Please reload this page.