-
Notifications
You must be signed in to change notification settings - Fork 157
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
[ros2node] Add option to info verb to display hidden names #373
Conversation
Changes behavior so that hidden names are not shown by default. Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Remove check for empty topic * Pass keyword arg by name Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Which it uses for identifier component containers. Signed-off-by: Jacob Perron <jacob@openrobotics.org>
@@ -38,28 +38,34 @@ def add_arguments(self, parser, cli_name): | |||
'node_name', | |||
help='Node name to request information') | |||
argument.completer = NodeNameCompleter() | |||
parser.add_argument( | |||
'--include-hidden', action='store_true', | |||
help='Display hidden topics, services, and actions as well') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe a user wants to see hidden nodes but not hidden topics / services / actions - or the other way around? Just a thought...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I presume you mean them wanting to see hidden topics, but not hidden services / actions (or some combination). I thought about it, but I figured it's a rare case to want to see any of the hidden names, let alone a subset.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much cleaner output. LGTM thanks!
Changes behavior so that hidden names are not shown by default.
Before this change, if we ran turtlesim and retrieved the node info we'd see hidden topics and services related to the action server, e.g.
Now, we can get the same output if we add the
--include-hidden
option, but by default we don't see the hidden entities like/turtle1/rotate_absoulte/_action/cancel_goal
: