Skip to content

run manipulation planning e2e on self-hosted linux and mac - #4184

Merged
bogwi merged 1 commit into
danvi/feat/add-web_browser-e2etests-to-mac-cifrom
danvi/feat/move-manipulation-e2e-to-self-hosted
Sep 16, 2026
Merged

bogwi merged 1 commit into
danvi/feat/add-web_browser-e2etests-to-mac-cifrom
danvi/feat/move-manipulation-e2e-to-self-hosted

Conversation

@bogwi

@bogwi bogwi commented Sep 16, 2026

Copy link
Copy Markdown
Member

Add test_single_arm_plans_and_executes_through_control_coordinator and test_dual_arm_plans_and_dispatches_both_arms_through_control_coordinator to Mac CI.

@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@                                Coverage Diff                                @@
##           danvi/feat/add-web_browser-e2etests-to-mac-ci    #4184      +/-   ##
=================================================================================
+ Coverage                                          79.17%   79.42%   +0.25%     
=================================================================================
  Files                                               1471     1471              
  Lines                                             139520   139520              
  Branches                                           11988    11988              
=================================================================================
+ Hits                                              110458   110809     +351     
+ Misses                                             25659    25314     -345     
+ Partials                                            3403     3397       -6     
Components Coverage Δ
Tests 95.59% <100.00%> (+0.59%) ⬆️
Flag Coverage Δ
OS-ubuntu-24.04-arm 75.10% <100.00%> (+<0.01%) ⬆️
OS-ubuntu-latest 75.80% <100.00%> (ø)
Py-3.10 75.57% <100.00%> (+<0.01%) ⬆️
Py-3.11 75.78% <100.00%> (+<0.01%) ⬆️
Py-3.12 75.79% <100.00%> (+<0.01%) ⬆️
SelfHosted-Large 30.52% <100.00%> (-0.21%) ⬇️
SelfHosted-Linux 36.65% <100.00%> (+0.45%) ⬆️
SelfHosted-macOS 36.94% <100.00%> (+1.41%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...mos/e2e_tests/test_manipulation_planning_groups.py 85.71% <100.00%> (ø)

... and 22 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread dimos/e2e_tests/test_manipulation_planning_groups.py
@dimensionalOS dimensionalOS deleted a comment from greptile-apps Bot Sep 16, 2026
@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Sep 16, 2026
leshy
leshy previously approved these changes Sep 16, 2026
@leshy
leshy added this pull request to the merge queue Sep 16, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue because a pull request earlier in the stack was removed Sep 16, 2026
@bogwi
bogwi force-pushed the danvi/feat/move-manipulation-e2e-to-self-hosted branch from 692ae50 to 02e65ca Compare September 16, 2026 15:21
@github-actions github-actions Bot added ready-to-merge Required CI checks have passed on this PR and removed ready-to-merge Required CI checks have passed on this PR labels Sep 16, 2026
@bogwi
bogwi requested a review from leshy September 16, 2026 16:19
@bogwi
bogwi added this pull request to the merge queue Sep 16, 2026
An error occurred while trying to automatically change base from danvi/feat/add-web_browser-e2etests-to-mac-ci to main September 16, 2026 17:26
Merged via the queue into main with commit a9a49c3 Sep 16, 2026
36 of 50 checks passed
@bogwi
bogwi deleted the danvi/feat/move-manipulation-e2e-to-self-hosted branch September 16, 2026 17:48
@github-actions

Copy link
Copy Markdown
Contributor

Successfully created backport PR for release/0.0.14:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport release/0.0.14 PlzReview ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants