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

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

Open
twdragon opened this issue Jun 8, 2023 · 2 comments

Comments

@twdragon
Copy link
Collaborator

twdragon commented Jun 8, 2023

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

  • Merging CMake build scenarios (waiting for a solution from @flynneva)
  • Dynamic generation of ROS parameters during node startup (solved for ROS1, searching for a solution for ROS2)
  • Exposure of ROS node as just a wrapper over the internal driver library API
@flynneva
Copy link
Collaborator

flynneva commented Jun 11, 2023

@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
Copy link
Collaborator

flynneva commented Nov 5, 2023

@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 👍🏼

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

No branches or pull requests

2 participants