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

Make joint_states subscription QoS configurable; default to SensorDataQoS #179

Merged
merged 1 commit into from Oct 26, 2021

Conversation

adeschamps
Copy link
Contributor

This changes the subscription default from reliable to
best_effort. This is helpful when visualizing a robot that's running
on a wifi-connected device, and eliminates a warning when starting up
drivers on a robot that publishes with best_effort.

The subscription defaults to best_effort, but can be overridden with
the following config:

robot_state_publisher:
  ros__parameters:
    qos_overrides:
      /joint_states:
        subscription:
          reliability: reliable

…aQoS.

This changes the subscription default from `reliable` to
`best_effort`. This is helpful when visualizing a robot that's running
on a wifi-connected device, and eliminates a warning when starting up
drivers on a robot that publishes with `best_effort`.

The subscription defaults to `best_effort`, but can be overridden with
the following config:

```yaml
robot_state_publisher:
  ros__parameters:
    qos_overrides:
      /joint_states:
        subscription:
          reliability: reliable
```
Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

Looks good to me. I'll run CI next.

@clalancette
Copy link
Contributor

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

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.

None yet

3 participants