ROS2 Migration about sick_tim HOT 8 CLOSED

You mean, sick_tim has USB support and sick_scan doesn't, right? And you want to ask about porting sick_tim to ROS2?

If you want to prepare a pull request to port sick_tim to ROS2, I'd be happy to review and merge it!

from sick_tim.

No, this repo is in maintenance mode, and I'm not planning on doing further development on it. There is a more up-to-date fork of this repo here, which also supports ROS2: https://github.com/SICKAG/sick_scan_xd

from sick_tim.

I already opened an issue in the past asking for USB support, but they are not interested in maintaining it so I wanted to know if there was any plan to port/maintain it in this repository for ROS2.

Thanks!

from sick_tim.

Hi there

As @Noel215 mentioned, we prepared a PR to support SICK TIM with USB support in ROS2.

That's what we did:

• we used Stratom Inc. foxy-devel branch to bring up ROS2 support
• we applied ROS2 conventions about linters and we make them happy
• some warnings fixed due to -Wpedantic introduced by Stratom Inc.
• Tests passed in a TIAGo robot with a USB TIM 551 with ros2_navigation and rviz

from sick_tim.

I cancelled PR #100 and I opened #101 with a cleaner git history with the same aims as stated before.

Sorry for the noise

from sick_tim.

@mintar just pinging. Do you think #101 can be merged?

from sick_tim.

Yes, thanks for the ping. I just created a new branch humble, rebased your PR and merged it. BTW, we have a Tiago robot as well (but still on ROS Kinetic). :)

from sick_tim.

Ah! Nice to know.
Hope you enjoy our little creature... and this patch soon!

from sick_tim.