Skip to content

Streamline IL: unify sim episode control, simplify create_dataset#83

Merged
wilsonchenghy merged 1 commit into
mainfrom
claude/awesome-ride-ccdaxl
Jun 12, 2026
Merged

Streamline IL: unify sim episode control, simplify create_dataset#83
wilsonchenghy merged 1 commit into
mainfrom
claude/awesome-ride-ccdaxl

Conversation

@wilsonchenghy

Copy link
Copy Markdown
Contributor
  • SimLeRobotRecorder gains start_keyboard() and tick() methods so the episode flags/keyboard/rate-limiting loop is owned by the recorder rather than duplicated across every teleop script. finalize() now stops the keyboard listener automatically.
  • All three teleop scripts (keyboard_teleop, so101_keyboard_teleop, so101_leader_teleop) drop ~15 lines each of flag-handling boilerplate; the per-step block reduces to recorder.tick(action, state, images).
  • create_dataset() simplified to a single required root: Path parameter; the dead record_root branch and get_next_experiment_path_with_gap import are removed from schema.py. LeRobotSink updated to match.

https://claude.ai/code/session_01NFYSHfEmqNn7d75qCnzAEy

- SimLeRobotRecorder gains start_keyboard() and tick() methods so the
  episode flags/keyboard/rate-limiting loop is owned by the recorder
  rather than duplicated across every teleop script. finalize() now
  stops the keyboard listener automatically.
- All three teleop scripts (keyboard_teleop, so101_keyboard_teleop,
  so101_leader_teleop) drop ~15 lines each of flag-handling boilerplate;
  the per-step block reduces to recorder.tick(action, state, images).
- create_dataset() simplified to a single required root: Path parameter;
  the dead record_root branch and get_next_experiment_path_with_gap
  import are removed from schema.py. LeRobotSink updated to match.

https://claude.ai/code/session_01NFYSHfEmqNn7d75qCnzAEy
@wilsonchenghy wilsonchenghy merged commit 18df1e6 into main Jun 12, 2026
1 check passed
@wilsonchenghy wilsonchenghy deleted the claude/awesome-ride-ccdaxl branch June 12, 2026 07:33
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.

2 participants