-
Notifications
You must be signed in to change notification settings - Fork 310
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
Add header to import limits from standard URDF definition #1298
Add header to import limits from standard URDF definition #1298
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1298 +/- ##
=======================================
Coverage 48.02% 48.02%
=======================================
Files 41 41
Lines 3525 3525
Branches 1912 1912
=======================================
Hits 1693 1693
Misses 442 442
Partials 1390 1390
Flags with carried forward coverage won't be shown. Click here to find out more. |
The failed test seems to be the
|
If it's fine, can we get this PR merged? |
(cherry picked from commit f3ffb56) # Conflicts: # joint_limits/CMakeLists.txt
(cherry picked from commit f3ffb56)
Add a helper header similar to the one of ROS1 : https://github.com/ros-controls/ros_control/blob/noetic-devel/joint_limits_interface/include/joint_limits_interface/joint_limits_urdf.h to import limits from the standard URDF model (no limits from ros2_control tag are retrieved)
I will parse these first and later in another upcoming PR, I will be parsing the limits from the ros2_control tag within the hardware_interface package