Unverified Commit b52df735 authored by Luis Carranza's avatar Luis Carranza Committed by GitHub

Fixes warning message in Articulation config processing (#699)

# Description

Fixes #698

The sanity check compares the total actuator joints (total_act_joints)
with the difference between the number of joints minus the number of
fixed tendons (self.num_joints - self.num_fixed_tendons), but the
warning message displays a comparison between total_act_joints and
self.num_joints. This is misleading on complex robots that includes
fixed tendons such as Franka hand.

## Type of change

- Bug fix (non-breaking change which fixes an issue)
parent 06c84fa1
...@@ -1008,7 +1008,7 @@ class Articulation(RigidObject): ...@@ -1008,7 +1008,7 @@ class Articulation(RigidObject):
if total_act_joints != (self.num_joints - self.num_fixed_tendons): if total_act_joints != (self.num_joints - self.num_fixed_tendons):
carb.log_warn( carb.log_warn(
"Not all actuators are configured! Total number of actuated joints not equal to number of" "Not all actuators are configured! Total number of actuated joints not equal to number of"
f" joints available: {total_act_joints} != {self.num_joints}." f" joints available: {total_act_joints} != {self.num_joints - self.num_fixed_tendons}."
) )
def _process_fixed_tendons(self): def _process_fixed_tendons(self):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment