Conversation
Codecov Report❌ Patch coverage is
@@ Coverage Diff @@
## fix/openyam-learning-baseline #3315 +/- ##
=================================================================
+ Coverage 79.57% 79.60% +0.02%
=================================================================
Files 1498 1501 +3
Lines 141657 141887 +230
Branches 12130 12141 +11
=================================================================
+ Hits 112730 112947 +217
- Misses 25502 25517 +15
+ Partials 3425 3423 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes 🚀 New features to boost your workflow:
|
a8ab2d8 to
486d959
Compare
01aa5d0 to
18ffcc7
Compare
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
18ffcc7 to
cda4f1f
Compare
f6a2f76 to
2a46dfb
Compare
2a46dfb to
884a430
Compare
884a430 to
2ffca94
Compare
89cc9f6 to
b8a7241
Compare
b8a7241 to
0c8996a
Compare
0c8996a to
564d788
Compare
Greptile SummaryThis change adds a host-side LeRobot policy integration backed by an isolated Python runtime, including rollout lifecycle handling and joint-command publication. Two failures were reproduced. Standard wheel installations omit the isolated runtime manifest required to start LeRobot, so rollout startup raises Confidence Score: 2/5Not safe to merge because wheel-installed LeRobot rollouts cannot start and stopped rollouts can still publish robot commands. The reproduced packaging failure blocks the feature in standard deployments, while the reproduced stop race can issue commands after a stop request. Files Needing Attention: MANIFEST.in and dimos/imitation/policy/lerobot/python/dimos_lerobot/runtime.py
What T-Rex did
|
Contribution path
Problem
Solution
LeRobotPolicyModulecontract.API shape
RolloutControlSpec:preflight_rollout(),start_rollout(),stop_rollout(),rollout_status().How to Test
Exercise the isolated runtime contract without robot hardware:
cd dimos/imitation/policy/lerobot/python uv run --isolated --locked --group tests --with-editable ../../../../../ python -m pytestAt this layer, 398 affected host tests and 27 isolated-runtime tests passed In the previous cleanup. The final stack also passed 413 control/Quest regression tests and all 36 isolated policy/preparation tests.
Scoped type checks and commit hooks passed.
Physical rollout and cancellation remain hardware release checks; mock tests are not safety certification.
The repository-wide Clippy hook reaches an existing
dim_slamflake failure resolvinggit+file:../../../..; that unrelated flake is unchanged.AI assistance
Checklist