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

[FEATURE]: Upgrade Gazebo from Garden to Harmonic #170

Closed
evan-palmer opened this issue May 23, 2024 · 7 comments
Closed

[FEATURE]: Upgrade Gazebo from Garden to Harmonic #170

evan-palmer opened this issue May 23, 2024 · 7 comments
Labels
enhancement New feature or request mid priority This is a medium priority issue

Comments

@evan-palmer
Copy link
Collaborator

evan-palmer commented May 23, 2024

Feature Type

Changing existing functionality in the BlueROV2 driver

Problem Description

The current LTS (Jazzy) and Rolling use Gazebo Harmonic instead of Gazebo Garden.

Feature Description

Upgrade the main and jazzy branches to use this version of Gazebo.

Alternative Solutions

N/A

Additional Context

No response

@evan-palmer evan-palmer added enhancement New feature or request needs triage This issue needs triage support labels May 23, 2024
@evan-palmer evan-palmer changed the title [FEATURE]: Upgrade Gazebo from Garden to Fortress [FEATURE]: Upgrade Gazebo from Garden to Harmonic May 23, 2024
@evan-palmer evan-palmer linked a pull request May 23, 2024 that will close this issue
@evan-palmer evan-palmer self-assigned this May 23, 2024
@evan-palmer evan-palmer added mid priority This is a medium priority issue and removed needs triage This issue needs triage support labels May 23, 2024
@evan-palmer
Copy link
Collaborator Author

This is currently being blocked by mavlink/mavros#1958. Once merged, we can resolve this issue.

@evan-palmer evan-palmer removed their assignment Jun 26, 2024
@evan-palmer
Copy link
Collaborator Author

evan-palmer commented Jul 31, 2024

Harmonic is now the recommended version for Humble and Iron. Might be good to upgrade those first while waiting for Jazzy support

@amarburg
Copy link
Collaborator

amarburg commented Aug 1, 2024

#220 upgrades Rolling to Harmonic. I could break just the upgrade to Harmonic into a separate PR which you could propagate to Humble and Iron (leaving the Rolling / Noble - specific changes in the existing PR)

@evan-palmer
Copy link
Collaborator Author

I just double-checked this, and I was wrong. Hypothetically, the backport should still work (ros_gz is already being built from source to support Garden), but I'm contemplating discontinuing the ros_gz source build in future releases to reduce some of the maintenance and build times (and also make things more stable). If we move to using the ros_gz binaries in #220, then we will probably want to keep separate PRs.

@amarburg
Copy link
Collaborator

amarburg commented Aug 1, 2024

So here's a merge of the ROS-Gazebo compatibility matrix and the ROS-Gazebo-ros_gz matrix:

ROS2 Version Gazebo Version ROS2-Gazebo Compat ros_gz binaries?
Humble Fortress Recommended https://packages.ros.org/
Humble Garden Possible gazebo packages
Humble Harmonic Possible gazebo packages
Iron Fortress Recommended https://packages.ros.org/
Iron Garden Possible From source
Iron Harmonic Possible From source
Jazzy Garden Possible From source
Jazzy Harmonic Recommended https://packages.ros.org/
Rolling Fortress Recommended https://packages.ros.org/
Rolling Garden Possible From source
Rolling Harmonic Possible From source

Based on the spotty availability of binaries, what's the desirable endpoint for blue?

Here's where we are now:

ROS2 Version Gazebo Version ros_gz
Humble Garden From source
Iron Garden From source
Jazzy Harmonic (PR work-in-progress here) From source (but packages are available)
Rolling Garden (or Harmonic via PR #220) From source (packages available for Harmonic)

I assume Rolling is converging with Jazzy (i.e. Harmonic will soon be the preferred version)?

@evan-palmer
Copy link
Collaborator Author

Here is what I'm thinking:

ROS 2 Version Gazebo Version ros_gz binaries
Humble Harmonic packages.osrfoundation.org
Iron Harmonic packages.osrfoundation.org
Jazzy Harmonic packages.ros.org
Rolling Harmonic Open to discuss

The reason that I am undecided on Rolling is because it may be nice to switch to new versions of Gazebo as they come out, but I am open to discuss that. Here are the docs on the Iron/Humble/Jazzy Harmonic installation.

@amarburg
Copy link
Collaborator

amarburg commented Aug 2, 2024

Hm. So I'm stuck on how to implement the switching between ros_gz sources from a git-mergify point of view. How about:

  1. This PR switches the default to harmonic. Apply it to main (aka rolling) and propagate to jazzy iron and humble.
  2. Targeted commit to jazzy which removes the source build for ros_gz and instead installs from packages.ros.org
  3. Targeted commit to iron which removes the source build and instead installs from osrffoundation.org

This should work for iron and humble at least. Jazzy is still broken per #168 , and Rolling still broken per #220

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request mid priority This is a medium priority issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants