Skip to content

Krishna/feat/openarm mini v2 - #3396

Draft
KrishnaH96 wants to merge 3 commits into
krishna/feat/openarm-damiaofrom
krishna/feat/openarm-mini-v2
Draft

Krishna/feat/openarm mini v2#3396
KrishnaH96 wants to merge 3 commits into
krishna/feat/openarm-damiaofrom
krishna/feat/openarm-mini-v2

Conversation

@KrishnaH96

Copy link
Copy Markdown
Contributor

Contribution path

  • Small, safe change that does not need a tracking issue
  • Linked issue or discussion: DIM-XXX / #XXX / URL

Problem

Solution

How to Test

AI assistance

Checklist

  • I have read and approved the CLA.

Ports dimos/teleop/openarm_mini from cc/feat/openarm-mini at 1563ff1,
including the multi-turn encoder wrap and the CLI tools. Adapted for the
whole-body OpenArm follower: default follower joint names come from
openarm_arm_joints, and the sender-side clamp limits mirror the OpenArm
v2.0 URDF instead of the v1.0 measured limits.
…llower

mini-teleop-openarm drives both arms from two leader ports; the left and
right variants drive one side while the other arm holds. Leader joint N
maps to follower joint N with no wrist reordering, matching the OpenArm
2.0 conventions on both ends.
Soft arm gains and grippers removed from the topology, the exact
configuration validated on both real arms during the leader teleop
ladder. Drop or promote this commit before opening the PR.
@codecov

codecov Bot commented Aug 8, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
3547 1 3546 174
View the top 1 failed test(s) by shortest run time
dimos.hardware.whole_body.openarm_damiao.test_adapter::test_openarm_topology_connects_arms_and_grippers
Stack Traces | 0.221s run time
openarm_adapter = <dimos.hardware.whole_body.openarm_damiao.adapter.OpenArmDamiaoAdapter object at 0xffe1b0c645f0>

    def test_openarm_topology_connects_arms_and_grippers(
        openarm_adapter: OpenArmDamiaoAdapter,
    ) -> None:
        robot = openarm_adapter._build_robot()
    
>       assert robot.group_names() == ["left_arm", "right_arm", "left_gripper", "right_gripper"]
E       AssertionError: assert ['left_arm', 'right_arm'] == ['left_arm', ...ight_gripper']
E         
E         Right contains 2 more items, first extra item: #x1B[0m#x1B[33m'#x1B[39;49;00m#x1B[33mleft_gripper#x1B[39;49;00m#x1B[33m'#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
E         
E         Full diff:
E         #x1B[0m#x1B[90m #x1B[39;49;00m [#x1B[90m#x1B[39;49;00m
E         #x1B[90m #x1B[39;49;00m     'left_arm',#x1B[90m#x1B[39;49;00m
E         #x1B[90m #x1B[39;49;00m     'right_arm',#x1B[90m#x1B[39;49;00m
E         #x1B[91m-     'left_gripper',#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
E         #x1B[91m-     'right_gripper',#x1B[39;49;00m#x1B[90m#x1B[39;49;00m
E         #x1B[90m #x1B[39;49;00m ]#x1B[90m#x1B[39;49;00m

openarm_adapter = <dimos.hardware.whole_body.openarm_damiao.adapter.OpenArmDamiaoAdapter object at 0xffe1b0c645f0>
robot      = <can_motor_control.Robot object at 0xffe1b10dd330>

.../whole_body/openarm_damiao/test_adapter.py:51: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

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