-
-
Notifications
You must be signed in to change notification settings - Fork 75
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
Add various: four-wheel-steering-msgs; ira-laser-tools; libuvc-camera; moveit-chomp-optimizer-adapter; roboticsgroup-upatras-gazebo-plugins; sick-tim; spatio-temporal-voxel-layer; urdf-geometry-parser required for tiago_public_ws installation on ROS1 #510
base: main
Are you sure you want to change the base?
Conversation
@FrancescoD98 - please remove the "ros-noetic-" prefix. |
On Linux there are some failures:
This is because we do not have a qt6-devel package (xref: RoboStack/robostack.github.io#65), the easy fix is to add the required library to pcl dependencies. |
Actually the root issue is another, it is the one fixed in SteveMacenski/spatio_temporal_voxel_layer#300 for ROS 2, but it was not fixed for ROS 1 . |
@FrancescoD98 can you use a more descriptive title for the PR? See for example RoboStack/ros-jazzy#19 . |
Thanks @FrancescoD98 ! |
The latest error is:
I am not sure if |
Yes, it seems that https://github.com/SteveMacenski/spatio_temporal_voxel_layer/blob/noetic-devel/cmake/FindTBB.cmake is quite old, probably we need to adapt it (for example using the ROS 2 branch as an example). @FrancescoD98 are you intersted in looking into it? |
@traversaro I'm still interested otherwise I cannot install the repo in which there is the robotic system that I have to use for my project. |
Great. See https://robostack.github.io/Contributing.html for indication on how to prepare patches, in your case I would first try to get rid of |
No description provided.