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

Creating a Subscriber with TopicStatistics Applies Custom QoS to /statistics Publisher #2314

Closed
JWhitleyWork opened this issue Sep 21, 2023 · 3 comments

Comments

@JWhitleyWork
Copy link

Bug report

Required Info:

  • Operating System:
    • Ubuntu 22.04
  • Installation type:
    • binaries
  • Version or commit hash:
    • Humble, Iron
  • DDS implementation:
    • Any
  • Client library (if applicable):
    • rclcpp

Steps to reproduce issue

rclcpp::SubscriptionOptions opts{};
opts.topic_stats_options.state = rclcpp::TopicStatisticsState::Enable;

rclcpp::QoS bad_qos{};

this->creare_subscription<DummyMsg>("/some_topic", bad_qos.history(rclcpp::HistoryPolicy::Unknown), cb_func, opts);

Expected behavior

The /statistics publisher has a stable, well-defined QoS profile.

Actual behavior

The QoS profile passed to create_subscription is applied to both the subscription and the /statistics publisher. Code reference in humble and iron.

@fujitatomoya
Copy link
Collaborator

@JWhitleyWork that makes sense to me. adding QoS to TopicStatisticsOptions struct as statistics publisher QoS configuration.

@CursedRock17
Copy link
Contributor

Since that PR was merged this issue should be complete right?

@clalancette
Copy link
Contributor

Yeah, I'm going to close it. Thanks!

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

No branches or pull requests

4 participants