Skip to content

Add --topic-config to specify QoS by topic via a .js file#2

Open
kaikoehn wants to merge 3 commits intodevelopfrom
kai/topic-config-file-param
Open

Add --topic-config to specify QoS by topic via a .js file#2
kaikoehn wants to merge 3 commits intodevelopfrom
kai/topic-config-file-param

Conversation

@kaikoehn
Copy link
Collaborator

@kaikoehn kaikoehn commented Sep 11, 2020

Same as in https://github.com/Intermodalics/im-vps-web/pull/174

We have another PR incoming solving another issue (related to managing subscribers with multiple clients), hence good to separate them as issues.

@renehoelbling
Copy link

Solves #1

@meyerj meyerj removed their request for review September 23, 2020 23:54
@meyerj
Copy link

meyerj commented Jan 8, 2021

Please consider to open a pull request upstream: https://github.com/RobotWebTools/ros2-web-bridge/pulls

@meyerj
Copy link

meyerj commented Feb 20, 2022

I think there are DDS vendor specific ways to overwrite the QoS settings in general or per topic using an xml file and by setting an environment variable (e.g. https://docs.ros.org/en/galactic/Tutorials/FastDDS-Configuration/FastDDS-Configuration.html). So this patch may not be required anymore.

@kaikoehn
Copy link
Collaborator Author

If we can solve it like this would be great! Getting our PR upstream, is probably harder.

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.

3 participants