Skip to content

Fix LIBERO multiview camera resolution - #40

Merged
wayrise merged 1 commit into
mainfrom
fix/libero-multiview-camera
Sep 28, 2026
Merged

wayrise merged 1 commit into
mainfrom
fix/libero-multiview-camera

Conversation

@wayrise

@wayrise wayrise commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Summary

LIBERO multiview configs can include target_camera for single-view deployment, but the reader previously treated it as an unconditional override and dropped the wrist stream. This PR resolves cameras from the dataset's advertised features in multiview mode while preserving single-view target-camera behavior.

The reader now supports the canonical image/image2 names and legacy agentview_image/wrist_image / robot0_eye_in_hand_image names, and synchronizes the resolved camera layout used during decoding.

Validation

  • Real LIBERO sample: head and wrist tiles decode with nonzero content; only the configured null tile is black.
  • pytest -q tests/dataloader/test_libero.py — 8 passed
  • pytest -q tests/dataloader tests/benchmarks — 657 passed, 4 skipped (optional labtasker dependency)
  • git diff --check — passed

@xueminchi
xueminchi self-requested a review September 28, 2026 15:39
@wayrise
wayrise merged commit 1204dde into main Sep 28, 2026
3 checks passed
@wayrise
wayrise deleted the fix/libero-multiview-camera branch September 28, 2026 15:45
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