-
Notifications
You must be signed in to change notification settings - Fork 13
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
Cpp code genrators for ROS2 nodes #130
Conversation
266b844
to
1c73bc0
Compare
find_package(sensor_msgs REQUIRED) | ||
|
||
add_executable(test_node src/test_node.cpp) | ||
ament_target_dependencies(test_node std_srvs sensor_msgs ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rclcpp is missing I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated to add rclcpp 76dc766#diff-d93fff18e9ce55d7368dd228b37f783b3a3821a4e16da3885bb636a29a43819eR97
… package to hold the generated code
I updated here also the ROS1 code generator to generate also a full package. For testing I added the autogenerated example to: https://github.com/ipa-nhg/test_ros_code_extractor and I am using the cpp code that we are generating to test the extractors: ipa320/ros-model-extractors@dbfc5a6#diff-3722d9ba8feb2d3feac8ce71a209a638d4b404e1c53f937188761181594023e2R23 All the tests are successful: https://github.com/ipa320/ros-model-extractors/actions 👏🏽 👏🏽 @ipa-hsd thanks for your help! should we merge this version and continue the improvements of the ROS2 code generation in #135 ? |
LGTM let's merge this one! |
These changes creates for the RosModel model:
the following ROS package:
https://github.com/ipa-nhg/test_ros2_code_extractor/tree/master/test_ros2
To test it I used the
ros-model-extractors
and its foxy docker container with the commands:The resulted model was: