-
Notifications
You must be signed in to change notification settings - Fork 40
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
Split into ROS and non-ROS repositories? #35
Comments
We should move forward with the changes to |
@marip8 It looks like we're halfway through this, since What do you think would be a good way to handle
I was originally thinking that some parts (e.g. the data_set library) could be moved into the non-ROS part of this repo, but on further investigation they ultimately depend on ROS parameter loading so that might not be feasible. For the time being I'm going to make a separate repo for my current ROS2 port of |
I think we could move the data and target loading functions into |
I've been using robot_cal_tools with ROS2, and I would like to contribute back some of the work I did in my weird fork to enable this without adversely affecting ROS1 projects that use this repo. Since some of the RCT packages contain ROS nodes, I think we will need to reorganize the structure of this repo.
A reasonable initial approach could be:
rct_optimizations
andrct_image_tools
pure CMake packages.rct_ros_tools
andrct_examples
to a new repository with parallel branches for ROS1 and ROS2.I recall that one of the goals of this project is to integrate it with the
industrial_calibration
project, withrobot_cal_tools
providing the underlying optimization libraries andindustrial_calibration
providing ROS nodes. I think restructuring this repo a little bit will actually help with that integration process down the road.The text was updated successfully, but these errors were encountered: