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

QoS conflict with joystick_drivers package #78

Open
roncapat opened this issue May 4, 2022 · 3 comments
Open

QoS conflict with joystick_drivers package #78

roncapat opened this issue May 4, 2022 · 3 comments

Comments

@roncapat
Copy link

roncapat commented May 4, 2022

reliability=rclpy.qos.QoSReliabilityPolicy.RELIABLE,

I run joystick_drivers's joy node, that publishes on topic /joy with BEST_EFFORT QoS setting.
However, this node subscribes with RELIABLE QoS setting, thus the subscription won't work.

Since the packages that modifies (see above) the default QoS is this one, I open an issue on this repo.

If you want, I can test and push a PR where subscription reliability is parametrized. Let me know if there are alternative solutions.

EDIT: sorry, now for some reason I can see that both pub and sub are RELIABLE

@roncapat roncapat closed this as completed May 4, 2022
@bmagyar
Copy link
Member

bmagyar commented May 5, 2022

I think we should always use best effort here

@roncapat
Copy link
Author

roncapat commented May 5, 2022

I'm not sure about the safety - I mean, may starting to loose incoming packets bring to an unsafe situation?

And/or

Shall we reopen the issue then to leave track of the needed change, if safe & needed?

@bmagyar
Copy link
Member

bmagyar commented May 6, 2022

Let's leave it open with the hopes of getting more philosophers joining the discussion and eventually reaching a conclusion

@bmagyar bmagyar reopened this May 6, 2022
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

2 participants