feat(manipulation): add Dual OpenYAM teleoperation - #3463
Conversation
❌ 5 Tests Failed:
View the top 3 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
e084e25 to
d9e5a17
Compare
40ef093 to
832a94b
Compare
Greptile SummaryThis change adds Dual OpenYAM model loading, dual-CAN hardware configuration, coordinated planning, and Quest teleoperation support. The integration test cannot load because it imports a removed mesh utility. The default model test selection also requires a large LFS-backed model that the regular job rejects, and the physical two-CAN configuration passes a runtime field that the Damiao adapter configuration rejects. T-Rex validation blockedFull execution of the model test path is blocked by missing Python packages Confidence Score: 2/5The change is not ready to merge because integration testing cannot load, the regular test job cannot retrieve its required model asset, and physical CAN configuration cannot create its adapter. Three independent non-security failures remain: the invalid mesh utility import, the unmarked guarded LFS model test, and the rejected physical-adapter configuration field. Files Needing Attention: dimos/robot/manipulators/dual_openyam/test_integration.py, dimos/robot/manipulators/dual_openyam/test_model.py, dimos/robot/manipulators/dual_openyam/config.py
What T-Rex did
|
832a94b to
883606d
Compare
ff35402 to
1bb53dd
Compare
0c6883b to
883606d
Compare
Summary
Testing
uv run dimos run teleop-quest-dual-openyamstarts successfully with mock hardwareStack
./bin/lfs_push