Skip to content
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

Auto-refresh graph #5

Open
dirk-thomas opened this issue Apr 24, 2017 · 3 comments
Open

Auto-refresh graph #5

dirk-thomas opened this issue Apr 24, 2017 · 3 comments

Comments

@dirk-thomas
Copy link
Contributor

From @danielsnider on December 11, 2016 18:24

I think rqt_graph would be even better if it had the option to automatically refresh the graph view, showing changes as they happen. I want to be able to monitor rqt_graph to see what is running at any given time without continually clicking refresh.

  • Do you like the auto-refresh idea?
  • Do you think auto-refresh should be the default behaviour?
  • Do you think auto-refresh should result in the refresh button being removed?
  • Do you think auto-refresh should be triggered by nodes state changes or a refresh rate in hz?

I might create a pull request if there are no major concerns.

Thanks!

Copied from original issue: ros-visualization/rqt_common_plugins#411

@dirk-thomas
Copy link
Contributor Author

Having the option to automatically refresh sounds good.

Since there are still use cases where that is not desired the manual mode should still be possible. I am not sure which default mode makes more sense but if that setting is restored on next startup it becomes a one time change for the user if he prefers the opposite.

Since there is no notification mechanism in ROS i think you will need to poll the information in regular intervals.

@dirk-thomas dirk-thomas changed the title rqt_graph: Auto-refresh graph Auto-refresh graph Jul 18, 2017
@EliteMasterEric
Copy link
Contributor

EliteMasterEric commented Aug 3, 2017

Information would have to be pulled at regular intervals, the list of nodes and edges checked against the current list, and refreshed if changed.

The option might need to be disabled by default unless there are performance improvements; when I run rqt_graph, it soft-locks the program for several seconds before it finishes processing and displays the new graph.

@Combinacijus
Copy link

+1 auto refresh would be nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants