ros-drivers / usb_cam

A ROS Driver for V4L2 USB Cameras
https://ros-drivers.github.io/usb_cam/main/
Other
536 stars 593 forks source link

[Community assistance needed] Merging ROS 1 and ROS 2 code over the same internal API #255

Open twdragon opened 1 year ago

twdragon commented 1 year ago

This issue is created to encourage any assistance in merging ROS1 and ROS2 branches. I want also to kindly ask @flynneva as a maintainer of ROS2 branch to write here the links to his studies about the merge, especially about CMake solution he uses. Please feel free to share your solutions. The huge work is already done, so the merging process should start!

Current general milestones

flynneva commented 1 year ago

@twdragon so the basic idea for this was already outlined in #203.

We can take the work I did last year, which is still available here and update the CI to work with the code that's been updated since then.

I'll put together a quick PR tomorrow and see what else needs to be updated.

cc @knorth55

flynneva commented 10 months ago

@twdragon @knorth55 fyi I've merged the CI for both ROS 1 and ROS 2 into the ros2 branch and set the ros1 jobs to be allowed to fail with #294.

Now it is up to you or someone else in the community to finish updating the ros1 node on the ros2 branch. When you do we can remove that "allow failure" flag for the CI job 👍🏼