Skip to content

Updates for 24.04/Jazzy/Harmonic #92

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Conversation

andermi
Copy link
Collaborator

@andermi andermi commented Jun 14, 2025

No description provided.

@andermi andermi requested a review from hamilton8415 June 14, 2025 00:27
Copy link

github-actions bot commented Jun 14, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://osrf.github.io/mbari_wec/pr-preview/pr-92/

Built to branch gh-pages at 2025-08-21 18:42 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@andermi
Copy link
Collaborator Author

andermi commented Jun 14, 2025

Copy link
Collaborator

@hamilton8415 hamilton8415 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was determined to get this installed and check the instructions this week. So had to push the week a little. :) But, Success! And I made a list of issues that were unclear or need fixing in the installation tutorial, otherwise everything appears to work. I'll exercise it some and approve the relevant PR's throughout.

  • In the "Install from source" instructions (https://osrf.github.io/mbari_wec/pr-preview/pr-92/Tutorials/Install/Install_source/) the note to "follow instructions for Installing Gazebo with ROS" may be confusing as that link is informational only...

  • I think we need to remove the instructions from the tutorial for 22.04/Humble/Garden. Confusing to have them there. we should discuss if we intend to keep the 22.04/Humble/Garden option available, and how we'd document that...

  • Instruction refers to ros-humble-rmw-cyclonedds-cpp, should be ros-jazzy-rmw-cyclonedds-cpp

  • We could add a suggestion to add export RMCexport RMW_IMPLEMENTATION=rmw_cyclonedds_cpp to .bashrc

  • This "install Gazebo Harmonic by installing ros_gz from the ros-jazzy-* apt repos. See here for more info." could use some more descriptoins. Following the "here" link might cause people to follow in structions there instead of the instructions on the next line of our install tutorial.

  • On my brand new 24.04 install, python3 -m pip gave an error " /usr/bin/python3: No module named pip". sudo apt-get install python3-pip fixed it.A

  • On my new 24.04 installl "sudo pip3 install -U rosdep" complained "This environment is externally managed". and suggested a few solutions. I went with "--break-system-packages again, which then resulted in "Requirement already satisfied: rosdep in /usr/lib/python3/dist-packages (0.25.1)". So maybe rosdep is already present? Anyway, rosdep then seemed to work...

  • Instructions still refer to humble "source /opt/ros/humble/setup.bash ", should be "source /opt/ros/jazzy/setup.bash"

andermi and others added 8 commits August 7, 2025 17:01
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
Signed-off-by: Michael Anderson <anderson@mbari.org>
* Updating tutorials

* Updated tutorial

* Updated log tutorial

* update log output tutorial to add rosbag2; update batch mode sim to show csv in output file structure

Signed-off-by: Michael Anderson <anderson@mbari.org>

* fix markdown in output log tutorial

Signed-off-by: Michael Anderson <anderson@mbari.org>

---------

Signed-off-by: Michael Anderson <anderson@mbari.org>
Co-authored-by: Michael Anderson <anderson@mbari.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants