Add subscription options to odometry subscriptions (main)

[pele1410]

Basic Info

Info	Please fill out this column
Ticket(s) this addresses	N/A
Primary OS tested on	Ubuntu 22.04 docker container)
Robotic platform tested on	Gazebo simulation
Does this PR contain AI generated software?	No

---

Description of contribution in a few bullet points

Added rclcpp::SubscriptionOptions to the odometry topic subscribers to allow qos overrides in the configuration. We run our odometry topics with Unreliable QoS to prevent network issues that are common with high-rate, reliable DDS transmissions. This change allows us to override the Reliable QoS of the odom subscriptions to allow matching.

Description of documentation updates required from your changes

I would expect that this should be documented as a new configuration option, but I'm not entirely sure where that would happen.

Description of how this change was tested

Testing was done against the Jazzy branch with #4961

---

Future work that may be required in bullet points

N/A

For Maintainers:

• Check that any new parameters added are updated in docs.nav2.org
• Check that any significant change is added to the migration guide
• Check that any new features OR changes to existing behaviors are reflected in the tuning guide
• Check that any new functions have Doxygen added
• Check that any new features have test coverage
• Check that any new plugins is added to the plugins page
• If BT Node, Additionally: add to BT's XML index of nodes for groot, BT package's readme table, and BT library lists

[SteveMacenski]

CI failed unusually. I retriggered it to see if that resolves the issue.

In the meantime, please sign off your commits with DCO -- check out the failed DCO job for instructions how

[SteveMacenski]

@pele1410 the same test is failing repeatedly, I think there's a problem here. I just checked our nightly jobs and PRs I merged today and those builds have all passed, so I'm fairly confident this is introducing a subtle bug.

unknown file
C++ exception with description "cannot set or declare a parameter, or change the callback from within set callback" thrown in the test body.

I'm seeing a ticket with this mentioned ros2/rclcpp#2741. It looks like this touches upon a recently reported issue with rclcpp - which you would probably hit eventually with Jazzy as well (good thing we ran this across Nav2's extensive CI 😉 )

[pele1410]

Well that's unfortunate. I can see an easy way around re-creating the OdomSmoother when the odom_duration changes (add a setter). But the odom_topic change is trickier. I could see capturing the QoS Override properties on construction and storing them, then using them to recreate the subscriber in a new method when the odom_topic changes. But that seems bad in that it could mislead users since the qos_overrides wouldn't actually be allowed to be changed.

Or we wait for that issue/PR to be resolved, which I assume is the path forward here.

[SteveMacenski]

I think so, there is a PR open, so hopefully it shouldn’t be too long! I commented in the ticket that we ran into it as well in Nav2, maybe that’ll help push it forward a bit 🙂

[pele1410]

I see ros2/rclcpp#2741 has been completed and landed on Rolling. Any chance we get an updated test here?

[SteveMacenski]

Has that been released into the binaries? I just retriggered CI, but its not been pushed into rolling binaries yet, then this won't work until it is released and available.

[SteveMacenski]

@pele1410 can you pull in main's updates? I think there's a caching related issue preventing the builds to all succeed since this is out of date

[SteveMacenski]

This throws an exception still:

C++ exception with description "cannot set or declare a parameter, or change the callback from within set callback" thrown in the test body.

Is this not out into binaries yet?

[pele1410]

Is this not out into binaries yet?

I'll be honest, I'm not entirely sure how to tell.

[pele1410]

It looks like it's in 29.5.0 based on this commit which was made 14 hours ago
ros2/rclcpp@11afffa

This might have been just after we re-ran the tests

[SteveMacenski]

The last rolling release was April 16th https://discourse.ros.org/t/new-packages-for-ros-2-rolling-ridley-2025-04-16/43256 which released 29.4.0 which was cut 3 weeks ago. Your PR was merged in 2 weeks ago, so its not available in binaries yet, but should be the next sync in approximately 2-3 weeks