-
Notifications
You must be signed in to change notification settings - Fork 72
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
Consider getting ROS1 message definition from topic connection header #33
Comments
Do we need to have the schema available when advertising? Can we advertise with just the topic name, then provide the schema content immediately after the client subscribes? |
The current ws-protocol spec requires the schema to be known when the channel is advertised. Not sure if we can / want to change this, I guess this would require also quite some changes to Studio. |
Yeah, we can evolve the protocol in a backwards compatible way if we need to though (require newer version of studio when using ros-foxglove-bridge). Just an idea if we want to avoid "open up 1000 tcp sockets" on startup. |
Studio needs the full set of schemas along with topic+datatype list when it connects to a data source. Rearchitecting Studio to fetch schemas on demand would be a large UX and technical undertaking. I don't think we need to avoid opening up 1000 TCP sockets on startup. I would expect it to take less than one second, happen before any WebSocket clients connect, and have a negligible impact on even resource-constrained systems while it is happening. |
Originally posted by @jhurliman in #29 (comment)
The text was updated successfully, but these errors were encountered: