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
Default to harmonic #54
base: main
Are you sure you want to change the base?
Conversation
* And clean up instructions * rosdep now resolves correctly Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@pulak-gautam Any chance you could test these changes? |
<exec_depend>ros_gz_sim</exec_depend> | ||
<exec_depend>ros_gz_bridge</exec_depend> | ||
<!-- TODO this installs ignition --> |
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.
Do we want to keep this if rosdep
installs ignition? It may be better to refer users to the Gazebo Sim documentation for running non-standard ROS 2 / Gazebo combos: https://gazebosim.org/docs/latest/ros_installation.
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.
I was not sure. If there were other rosdep keys, I would say yes, but I haven't found them. We still want to ensure the build order is correct. Perhaps I should ask on the discord server.
wget https://raw.githubusercontent.com/ArduPilot/ardupilot_gz/main/ros2_gz.repos | ||
vcs import --recursive < ros2_gz.repos | ||
cd ~/ros2_ws | ||
mkdir src |
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.
Don't need this as created above in L31.
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.
Looks good. A few questions in the review notes.
Purpose
harmonic