Skip to content

Fix type casting for wp.argmax return value in GripperCreator#6

Open
myurasov-nv wants to merge 1 commit intoNVlabs:mainfrom
myurasov-nv:fix/gripper-axis-type-cast
Open

Fix type casting for wp.argmax return value in GripperCreator#6
myurasov-nv wants to merge 1 commit intoNVlabs:mainfrom
myurasov-nv:fix/gripper-axis-type-cast

Conversation

@myurasov-nv
Copy link
Copy Markdown

@myurasov-nv myurasov-nv commented Mar 27, 2026

Description

wp.argmax() returns a warp type that cannot be used directly as a Python int for indexing and arithmetic. This causes runtime errors in GripperCreator._find_grasp_parameters() when computing open_axis, approach_axis, and mid_axis.

Changes

  • Wraps wp.argmax() calls with int() in scripts/graspgen/create_gripper_lab.py to ensure proper type for subsequent indexing and arithmetic operations.

Testing

@myurasov-nv myurasov-nv marked this pull request as ready for review March 27, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant