You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Implementing a service features could simplify implementing certain dataflows, including supporting ROS 2 legacy applications.
API details
I think that we can actually build around our current pub-sub pattern to support request-reply.
What we could do is to make a new method called send_request or request that will send an output and receive an input in one blocking function call. The function will return the reply from the request.
To do that, we could add the following dataflow pattern:
The webcam code does need to be changed. And the underlying communication design does not need to change as well so we can still make it flexible.
Edge cases details
We should try to keep message that has been received in between the request-reply, so that to not miss out on important messages. And so some work has to be done within the message queue.
Open for discussions
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
Implementing a
service
features could simplify implementing certain dataflows, including supporting ROS 2 legacy applications.API details
I think that we can actually build around our current pub-sub pattern to support request-reply.
What we could do is to make a new method called
send_request
orrequest
that will send an output and receive an input in one blocking function call. The function will return the reply from the request.To do that, we could add the following dataflow pattern:
With the following python operator request code:
With the following python custom node request code:
The webcam code does need to be changed. And the underlying communication design does not need to change as well so we can still make it flexible.
Edge cases details
Open for discussions
The text was updated successfully, but these errors were encountered: