Add Graph description messages to rosgraph_msgs#188
Conversation
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
dc66d05 to
7fea0f1
Compare
skyegalaxy
left a comment
There was a problem hiding this comment.
seems like pretty much exactly what was discussed at this week's ROSgraph WG
skyegalaxy
left a comment
There was a problem hiding this comment.
need to add DCO for the last commit
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
31aa0e1 to
0892e87
Compare
|
Note for reviewers - I'm updating the docstrings with more information, and calling out the change here. I was trying to use these messages to represent both the "abstract descriptor" of these entities, as well as the "observed/observable instantiation" of them. However, those two concepts have some key disjunctions. Therefore, I am focusing in on the latter - observed/observable instantiation. We will use NoDL for the "abstract descriptor" and may or may not choose to be able to represent it as messages, which will be a separate exercise. Thanks @fujitatomoya for taking the close look that forced this decision |
…ns specifically to observed/actual state, rather than trying to fold in an abstract descriptor meaning Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
| Service[] service_servers | ||
|
|
||
| Action[] action_clients | ||
| Action[] action_servers |
There was a problem hiding this comment.
Would these lists allow for duplicates, say for example if a node spins up multiple (action) clients to the same (action) server for whatever reason?
There was a problem hiding this comment.
nothing prevents it. based on the current representation you wouldn't be able to tell them apart though.
I'm now wondering if I need to basically have the leaf message actually be an encoding of rmw_topic_endpoint_info (and service_endpoint_info), and include the GID as well.
That would directly represent the "observable state" as it's defined most fundamentally in the API, if we try to match those definitions.
|
@emersonknapp is this ready to merge? I can run CI |
|
Pulls: #188 |
|
CI failures unrelated, merging |
|
@Mergifyio backport humble jazzy kilted |
✅ Backports have been createdDetails
|
Add new message definitions to rosgraph_msgs that can be used to describe a running or potential ROS Graph. Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com> (cherry picked from commit 2209d02)
Add new message definitions to rosgraph_msgs that can be used to describe a running or potential ROS Graph. Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com> (cherry picked from commit 2209d02)
Add new message definitions to rosgraph_msgs that can be used to describe a running or potential ROS Graph. Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com> (cherry picked from commit 2209d02)
Add new message definitions to rosgraph_msgs that can be used to describe a running or potential ROS Graph. (cherry picked from commit 2209d02) Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com> Co-authored-by: Emerson Knapp <537409+emersonknapp@users.noreply.github.com>
Add new message definitions to rosgraph_msgs that can be used to describe a running or potential ROS Graph. (cherry picked from commit 2209d02) Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com> Co-authored-by: Emerson Knapp <537409+emersonknapp@users.noreply.github.com>
Description
Fixes #182
Add new message definitions to
rosgraph_msgsthat can be used to describe a running or potential ROS Graph.Graph: a list of nodesNode: describe a single node's parameters and interface endpointsTopic: describe a single topic endpoint, either Publisher or SubscriptionService: describe a single service endpoint, either Server or ClientAction: describe a single action endpoint, either Server or ClientQoSProfile: describe a QoS Profile, either offered or requestedInterfaceType: identify arosidltypeTypeHash: identify a REP-2016 RIHS type hash (rosidl_runtime_cdefined type)This is intended to supplant the definitions in use at https://github.com/ros-tooling/graph-monitor/tree/main/rosgraph_monitor_msgs
Is this user-facing behavior change?
Yes
Did you use Generative AI?
No
Additional Information
Part of the NoDL and rosgraph_monitor work being done within the ROSGraph Working Group.
We would like to get these messages into the core for Lyrical, so that we can build tools around them for release into that LTS distro.
These messages act as a counterpart to the NoDL schema, being used to output either a desired or observed state of a ROS application as a message between nodes. The NoDL schema is not a strict duplicate of the information that these messages will contain, because: