Skip to content

test(robocasa): move simulator smoke checks into the E2E suite (v0.1) - #269

Merged
Sonorous281 merged 4 commits into
RLinf:mainfrom
ye1457:test/robocasa-smoke-layout
Oct 14, 2026
Merged

Sonorous281 merged 4 commits into
RLinf:mainfrom
ye1457:test/robocasa-smoke-layout

Conversation

@ye1457

@ye1457 ye1457 commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

Changes

Move four existing RoboCasa simulator smoke checks into tests/e2e_tests/robocasa/test_environment_smoke.py and enable them in the existing GPU runner. The test file is unchanged; the runner now executes seven checks in one suite.

Basic initialization/rendering checks overlap with the environment component test. The smoke tests also cover navigation RGB-D/world maps, mobile-camera tracking, and the NavigateKitchen and PickPlaceCounterToCabinet scenes. The component and policy-chain tests cover RPC observations, RLDX inference, and CLI-to-action execution. The README now distinguishes these responsibilities and their resources. Both RoboCasa guides use the new test path.

Validation

  • Standalone smoke: 4 passed without model/output paths; 4 skipped when disabled.
  • Existing GPU runner in a fresh Python 3.10 environment: 7 passed, 0 skipped; no remaining daemon processes.
  • Full unit suite in a separate unit-test environment: 1280 passed, 7 skipped.
  • Pre-commit, strict English/Chinese Sphinx builds, and rendered command/link checks passed.

GPU checks reused local weights/assets and cached package versions, including Torch 2.7.0+cu126 and torchvision 0.22.0+cu126. Unit checks used CI's OpenCV 4.10 dependency. Both environments passed uv pip check.

The earlier mixed environment exposed a 150 ms timeout-test failure during a 213 ms garbage-collection pause. The test passed unchanged in the separate unit-test environment; the failure and diagnostics are retained locally. No test thresholds or dependency constraints were changed in the repository.

Rebased onto main ef70215. No new CI workflow is added. Runtime and memory behavior are unchanged.

@read-the-docs-community

read-the-docs-community Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

@read-the-docs-community

read-the-docs-community Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

@ye1457 ye1457 changed the title test(robocasa): move simulator smoke checks into the E2E suite test(robocasa): move simulator smoke checks into the E2E suite (v0.1) Oct 13, 2026
@wilburx813

Copy link
Copy Markdown
Collaborator

should check whether it is duplicated with previous ci

@ye1457
ye1457 force-pushed the test/robocasa-smoke-layout branch from d099ef7 to d9bbc9a Compare October 13, 2026 14:17
@ye1457

ye1457 commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator Author

Checked against the existing GPU suite. Initialization/rendering overlaps, but it does not cover navigation RGB-D/world maps, mobile-camera tracking, or the NavigateKitchen and PickPlaceCounterToCabinet scenes. I kept those checks and clarified the README. All seven cases run once through the existing GPU runner; no new workflow was added.

Rebased onto ef70215; updated in d9bbc9a. Standalone smoke: 4 passed. Full GPU suite: 7 passed, with no skips or leftover daemons. The isolated unit-test run passed 1280 tests, with 7 skipped. The PR description records an earlier timing-test failure in the mixed environment.

Comment thread tests/README.md Outdated
@Sonorous281
Sonorous281 merged commit 2ba861e into RLinf:main Oct 14, 2026
7 checks passed
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.

3 participants