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

px4_ros_com reorganization #15

Merged
merged 3 commits into from
Jan 20, 2019
Merged

px4_ros_com reorganization #15

merged 3 commits into from
Jan 20, 2019

Conversation

TSC21
Copy link
Member

@TSC21 TSC21 commented Jan 20, 2019

This reorganisation pull request has the following purposes:

  1. Have a specific PX4 ROS messages package (px4_msgs), which are auto generated on the build process. At the same time, this package also generates the interfaces with ROS2 DDS;
  2. Have a specific px4_ros_com package, which generates and builds the micro-RTPS agent code based on also generated IDL files. It also contains ROS nodes examples to interface with PX4 using the interfaces from px4_msgs;
  3. This is also fixes Feature: remove yaml for ros2 and ros1 messages #13, since we will have px4_msgs on both ROS2 and ROS1 workspaces.

For now, this PR only reorganizes the master branch, meaning, the ROS2 branch. Following this, the ros1 branch will suffer the same reorganization.

@thomasgubler @jkflying FYI

@TSC21 TSC21 added the enhancement New feature or request label Jan 20, 2019
@TSC21 TSC21 merged commit a4c336b into master Jan 20, 2019
@TSC21 TSC21 deleted the ros2_package_reorganization branch January 20, 2019 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: remove yaml for ros2 and ros1 messages
1 participant