Skip to content

fix(lerobot): preserve full task lists when exporting multi-task episodes - #653

Closed
harshitethic wants to merge 2 commits into
Hebbian-Robotics:mainfrom
harshitethic:fix/lerobot-multitask-index-632
Closed

harshitethic wants to merge 2 commits into
Hebbian-Robotics:mainfrom
harshitethic:fix/lerobot-multitask-index-632

Conversation

@harshitethic

Copy link
Copy Markdown
Contributor

Summary

  • retain each source episode's full tasks list while reconstructing the materialized LeRobot corpus
  • validate frame task_index values against that full list instead of the collapsed provenance task
  • publish the same source task list in the exported episode metadata
  • add a regression test with frame indexes 0 and 1 resolving to two source tasks

The existing invalid-index regression remains meaningful: a frame still fails before output when its index is outside the source episode's published task list.

Fixes #632.

@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

👋 Hi @harshitethic — thanks for the contribution! To keep starter issues available
for other contributors and give every pull request a real review, we accept
1 open pull request per contributor at a time.

You already have #651 open, so this one is being closed automatically.
Once your open pull request is merged or closed, feel free to reopen this one —
no work is lost.

@github-actions github-actions Bot closed this Oct 1, 2026
@greptile-apps

greptile-apps Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Fixes task list handling in episode export logic.

The PR appears safe to merge; no actionable issue was established.

Summary

LeRobot export now preserves each selected episode’s full source task list and checks frame indexes against it.

  • Exported episode metadata keeps every source task.
  • Frames can keep indexes that point to any task in that list.

Reviews (1) · Last reviewed commit: "test(lerobot): cover multi-task source i..."

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.

[Bug]: examples/lerobot/export.py rejects task_index >= 1 on multi-task datasets during pre-validation

1 participant