Revert "Krishna/feat/openarm damiao" - #3387
Conversation
This reverts commit b7a63c2.
Greptile SummaryReverts the bimanual OpenArm whole-body integration in favor of separate v10 manipulator adapters, CAN drivers, model configurations, and blueprints. It also changes generic planning, mesh conversion, visualization, and inactive Damiao feedback behavior.
Confidence Score: 2/5This PR should not merge until mesh conversion preserves distinct geometry, same-robot planning groups remain composable, and inactive Damiao reads continue advancing feedback. The changes can substitute incorrect collision geometry during model preparation, reject previously supported coordinated group plans, and return frozen hardware state in read-only sessions. Files Needing Attention: dimos/manipulation/planning/utils/mesh_utils.py, dimos/manipulation/planning/world/roboplan_model.py, dimos/hardware/whole_body/damiao/adapter.py Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart LR
BP[OpenArm blueprint] --> L[Left HardwareComponent]
BP --> R[Right HardwareComponent]
L --> LA[OpenArmAdapter]
R --> RA[OpenArmAdapter]
LA --> LCAN[SocketCAN bus]
RA --> RCAN[SocketCAN bus]
BP --> P[Manipulation planner]
P --> LM[Left RobotModelConfig]
P --> RM[Right RobotModelConfig]
LM --> C[Coordinated planning]
RM --> C
Reviews (1): Last reviewed commit: "Revert "Krishna/feat/openarm damiao (#33..." | Re-trigger Greptile |
| mesh_name = Path(original_path).stem | ||
| path_tag = hashlib.md5(original_path.encode()).hexdigest()[:8] | ||
| obj_path = mesh_dir / f"{mesh_name}_{path_tag}.obj" | ||
| obj_path = mesh_dir / f"{mesh_name}.obj" |
| if len({group.robot_name for group in selected}) < 2: | ||
| continue |
There was a problem hiding this comment.
| # active; without it feedback would stay frozen at the connect | ||
| # snapshot, so keep it flowing for read-only sessions. | ||
| self._refresh() | ||
| states: list[MotorState] = [] |
There was a problem hiding this comment.
Reverts #3351