Closed saikishor closed 2 months ago
These changes make the JTC use the internal method to retrieve the URDF rather than using the variables directly. I think it is safer and clear this way.
Once the https://github.com/ros-controls/ros2_control/pull/1623 is merged, we would definitely need these changes. These changes can also be merged even without 1623.
Thank you!
Attention: Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.
50.00000%
2 lines
Project coverage is 86.55%. Comparing base (990bfb6) to head (01fc178).
990bfb6
01fc178
These changes make the JTC use the internal method to retrieve the URDF rather than using the variables directly. I think it is safer and clear this way.
Once the https://github.com/ros-controls/ros2_control/pull/1623 is merged, we would definitely need these changes. These changes can also be merged even without 1623.
Thank you!