ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

Tools: Updated ardupilot-ws and dependencies

Open snktshrma opened this issue 2 years ago • 4 comments

This PR is an update to #22598 and #22608 and will resolve all the dependency related issues with ardupilot_ros package. This ardupilot_ros package is tested with SwiftGust's and Khancyr's version of ardupilot_gazebo package for simulation purposes.

Also updated the navigation parameters ArduPilot/ardupilot_ros#5 in response to this PR.

Also I have added the script in comments to add packages.osrfoundation.org to the apt source to download gz-garden for ardupilot_gazebo.

There's an issue found while downloading gz-garden. It tries to remove ROS and gazebo packages because of conflicting dependencies. Not sure if it's for all.

snktshrma avatar Jan 10 '23 21:01 snktshrma

@rmackay9 @srmainwaring can you please have a look at the updates? Thank you

snktshrma avatar Jan 12 '23 17:01 snktshrma

This ardupilot_ros package is tested with SwiftGust's and Khancyr's version of ardupilot_gazebo package for simulation purposes.

Can we add a feature to specify which combination of ROS / Gazebo / ardupilot_gazebo to install. The ardupilot_gazebo plugin needs to be matched to an appropriate version of Gazebo and that in turn is matched to a ROS release for a version of Ubuntu.

  • https://www.ros.org/reps/rep-2000.html (ROS REP 2000 list the ROS versions supported on each version of Ubuntu)
  • https://classic.gazebosim.org/tutorials?tut=ros_wrapper_versions&cat=connect_ros (for legacy)
  • https://github.com/gazebosim/ros_gz (for next gen)

We should make sure that ArduPilot/ardupilot_gazebo has a branch for each of the Gazebo versions we are planning to support so the maintenance is in one place. If we can simplify this to one legacy version and to the next gen versions (from Garden onwards after the namespace change) that would simplify support.

srmainwaring avatar Jan 12 '23 20:01 srmainwaring

@snktshrma,

Thanks for this.

Can you rebase on master to remove the "Merge" commit? We don't allow merge commits to be merged to master.

rmackay9 avatar Jan 13 '23 00:01 rmackay9

@srmainwaring Thanks for the suggestion. I'll add an option for the users to select their combination of ardupilot_gazebo and other required packages. This will be good for users to have a customized and comfortable setup. @rmackay9 can you please have a look at the pr? Thank you

snktshrma avatar Jan 13 '23 20:01 snktshrma