Skip to content
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

Two and three D405 examples #999

Closed
wants to merge 4 commits into from

Conversation

adricort
Copy link

Since there are not well documented examples in ROS2 for multi cameras, I configured two launch codes with their respective easy-to-use rviz config files, which mainly are two examples with:

  • 2 Intel RealSense D405 cameras
  • 3 Intel RealSense D405 cameras
    In can manage up to 4, but I only tested it with 3.
    It took me some time and an open issue (it will be closed soon) in the forum to get it working in the real robotic arm that I am working on. This will help people to transfer quickly their projects.

I am using: multi cameras to create a single rgbd_image, which will help to create a synchronous MapCloud, and, so far, it is only using the odometry of the front camera.

Thanks to the team for the help, I hope the pull request can be accepted.

@SammyRamone
Copy link

This is the related issue #983

@matlabbe
Copy link
Member

As the name of the launch file is specific, could you remove all the conditional arguments, make hard-coded parameters that should be used by his example, and remove all arguments declared that are not used? Just make to most basic example like I did in the d435i example launch files: https://github.com/introlab/rtabmap_ros/tree/ros2/rtabmap_examples/launch

@adricort
Copy link
Author

Okay, if it is to be hard-coded, then I will. Thank you for the feedback!

@adricort
Copy link
Author

Closed in favor of #1002, see comment there.

@adricort adricort closed this Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants