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.

Sign up for GitHub

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

Jump to bottom

Tools: Install ROS 2 eProsima integration service profile #24280

Closed
wants to merge 2 commits into from
Closed

Tools: Install ROS 2 eProsima integration service profile #24280

wants to merge 2 commits into from

Conversation

srmainwaring
Copy link
Contributor

Install the AP_DDS integration service profile when building with colcon.

  • Allows config to be located using ros2 pkg tools in launch files.

Dependencies

@srmainwaring srmainwaring mentioned this pull request Jul 12, 2023
Merged
@srmainwaring srmainwaring force-pushed the prs/pr-ros2-install-is-config branch from 52bc072 to d176102 Compare July 19, 2023 09:03
@srmainwaring
AP_DDS: rename integration service profile
379ec11 
Signed-off-by: Rhys Mainwaring <rhys.mainwaring@me.com>
@srmainwaring
Tools: install ROS 2 eProsima integration service profile
a063660 
Signed-off-by: Rhys Mainwaring <rhys.mainwaring@me.com>
@srmainwaring
Copy link
Contributor Author

@Ryanf55 - renamed IS service profile to be consistent with #24706 and rebased.

Can check installation with:

colcon build --packages-select ardupilot_sitl

then

$ ls ./install/ardupilot_sitl/share/ardupilot_sitl/config
DDS_Service_IS_config.yaml	default_params
dds_xrce_profile.xml

@srmainwaring srmainwaring marked this pull request as draft September 5, 2023 18:10
@srmainwaring
Copy link
Contributor Author

@Ryanf55 - I'll hang fire on this - I've found a way to dispense with the need to use integration_service for the AP_DDS service calls, and think that will be a better option.

@srmainwaring srmainwaring mentioned this pull request Sep 5, 2023
Closed
@srmainwaring
Copy link
Contributor Author

Closed in favour of #24880

@srmainwaring srmainwaring deleted the prs/pr-ros2-install-is-config branch September 14, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ryanf55 Ryanf55 Ryanf55 approved these changes

Assignees
No one assigned
Labels
ROS
Projects
DDS/ROS2
Status: ✅ Done
GSoC-2023 ROS2 Gazebo
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@srmainwaring @Ryanf55