-
Notifications
You must be signed in to change notification settings - Fork 27
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
[galactic] Fix quiet filtering for node_topic graphs #70
[galactic] Fix quiet filtering for node_topic graphs #70
Conversation
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.
Alright, going in! Thanks for the fix @DLu. |
@DLu would you mind to sign off your commit before I merge? |
Signed-off-by: David V. Lu <davidvlu@gmail.com>
aa8f5ba
to
9b66e69
Compare
Done |
@hidmic How do you want to backport? I can open up new PRs if you want |
A backport PR for Foxy sounds good. We can run Foxy CI there. |
Signed-off-by: David V. Lu <davidvlu@gmail.com>
New version of PR #69 based on galactic-devel
Here's a standard graph with Debug/Quiet=False
However, when we turn on Debug/Quiet, we are not correctly filtering the edges.
This is an effect of the nodes being properly filtered, but the edges not being filtered, so new nodes with the dot node names (
t__rosout
) are created.The new logic always filters the edges (instead of just when
graph_mode == NODE_NODE_GRAPH
. We also always filter thent_nodes
, which is just empty ifgraph_mode == NODE_NODE_GRAPH
.@hidmic I think you wrote this bit of logic originally in #35