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

No way to set rosout QoS for a Node #1368

Open
rg-gravis opened this issue Oct 4, 2024 · 2 comments · May be fixed by #1376
Open

No way to set rosout QoS for a Node #1368

rg-gravis opened this issue Oct 4, 2024 · 2 comments · May be fixed by #1376
Labels
help wanted Extra attention is needed

Comments

@rg-gravis
Copy link

Feature request

Feature description

The rclcpp NodeOptions class has a rosout_qos field that allows you to set the QoS policy for when the node publishes to /rosout as part of ROS logging. The rclpy Node constructor has no such option, so python users are stuck with the default rosout_qos, which is unsuitable for some applications (we're experiencing dropped log messages that can be resolved for cpp nodes by adjusting the rosout_qos).

Implementation considerations

An additional parameter in the Node constructor would work perfectly well, and would just need to get passed to rcl here

@cottsay
Copy link
Member

cottsay commented Nov 1, 2024

Hi there, this seems like a reasonable feature to add. Please consider opening a pull request to implement it!

@cottsay cottsay added the help wanted Extra attention is needed label Nov 1, 2024
@fujitatomoya fujitatomoya linked a pull request Nov 3, 2024 that will close this issue
@fujitatomoya
Copy link
Collaborator

@rg-gravis you can try #1376.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants