ros2_control icon indicating copy to clipboard operation
ros2_control copied to clipboard

view_controller_chains does not work with example

Open christophfroehlich opened this issue 1 year ago • 4 comments

Describe the bug The output of view_controller_chains does not seem to work with the new example 12 of the demo repository.

To Reproduce Steps to reproduce the behavior:

  1. Checkout https://github.com/ros-controls/ros2_control_demos/
  2. Run the example 12 as written in the doc
  3. ros2 control view_controller_chains

Expected behavior Some chain in the PDF.

Screenshots But there are no links, just nodes: controller_diagram.gv.pdf

Environment (please complete the following information):

  • OS: Ubuntu 22.04
  • Version: Rolling

Additional context none

christophfroehlich avatar Aug 07 '23 06:08 christophfroehlich

@pac48 could you please have a look on that if you find time?

christophfroehlich avatar Aug 07 '23 06:08 christophfroehlich

I was able to reproduce the error that you mentioned. It seems that the Python library was changed from graphviz to pygraphviz in this PR https://github.com/ros-controls/ros2_control/pull/812 When I run with the old graphviz way, I get this output.
Screenshot from 2023-08-11 15-36-46 I am not too familiar with the differences between graphviz and pygraphviz, but it seems there is a difference in usage.

Is it possible to get a diagram of what the output should look like? The output above doesn't seem right.

pac48 avatar Aug 11 '23 19:08 pac48

I haven't seen the changes from your version, sorry. Maybe @sachinkum0009 can have a look?

@saikishor designed the example, imho the graph posted above isn't correct?

christophfroehlich avatar Aug 12 '23 09:08 christophfroehlich

I believe the motivation for using a different graphviz python implementation was that the former isn't supported by rosdep

bmagyar avatar Aug 12 '23 11:08 bmagyar