Skip to content

Commit

Permalink
[N/A] bugfix: give new xacro arguments default values (#521)
Browse files Browse the repository at this point in the history
## Change Overview

fixes an issue where new gripperless arguments needed to be specified for loading the arm URDF. now if they aren't specified, defaults are used instead of throwing an error. 

## Testing Done

- [x] standalone arm URDF can be loaded properly if these arguments aren't specifically set
  • Loading branch information
khughes-bdai authored Nov 6, 2024
1 parent f0da5d3 commit 3d23f53
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion spot_description/urdf/spot_arm_macro.urdf
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
<?xml version="1.0"?>
<robot name="spot_arm" xmlns:xacro="http://www.ros.org/wiki/xacro">
<xacro:macro name="load_arm" params="
tf_prefix gripperless custom_gripper_base_link">
tf_prefix
gripperless:=false
custom_gripper_base_link:=''">

<!-- Useful boolean variable for determining if custom gripper base link should be used in the xacro -->
<xacro:property name="use_custom_gripper_base_link" value="${gripperless and custom_gripper_base_link!=''}"/>
Expand Down

0 comments on commit 3d23f53

Please sign in to comment.