stereolabs / zed-ros-wrapper

ROS wrapper for the ZED SDK
https://www.stereolabs.com/docs/ros/
MIT License
441 stars 389 forks source link

Update wrapper #918

Closed Myzhar closed 10 months ago

Myzhar commented 10 months ago

Port to ROS Wrapper the main improvements from the ROS 2 Wrapper

Myzhar commented 10 months ago

That's a correct behavior. You must change Fixed frame to <camera_name>_camera_base_link or to base_link because map and odom frames are not published. Otherwise you can change the image viewer module to use the other one that does not require and existing frame id

mattrouss commented 10 months ago

Instead of changing the module entirely, would it be possible to set the frame to base_link only for the camera module? And will doing so impact the display of the other modules like object detection or skeleton tracking?

Myzhar commented 10 months ago

That's not the usual way of using RVIZ. I prefer to leave it to map which is good in any case. If a user disables depth RVIZ is not useful, there are other tools to visualize only images.

mattrouss commented 10 months ago

Can you update the camera viewer to not show the IMU pose by default ? It does not add much and is a bit disturbing

image

Myzhar commented 10 months ago

Yes

Myzhar commented 10 months ago

Modification pushed, but it's in the repository zed-ros-examples, not this