docs(il_robots): update API examples and bash snippets from SO100 to SO101#3409
Draft
TaliesinYang wants to merge 2 commits intohuggingface:mainfrom
Draft
docs(il_robots): update API examples and bash snippets from SO100 to SO101#3409TaliesinYang wants to merge 2 commits intohuggingface:mainfrom
TaliesinYang wants to merge 2 commits intohuggingface:mainfrom
Conversation
All bash commands in the il_robots tutorial use so101_follower and so101_leader, but the Python API examples referenced SO100Follower, SO100FollowerConfig, SO100Leader, and SO100LeaderConfig. Align the Python examples to match the SO101 hardware used in the CLI examples. Affected sections: Record a dataset, Replay an episode, Evaluate a policy. Fixes huggingface#3219 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Follow-up to the earlier commit: the Evaluate-a-policy section still had so100_follower / eval_so100 / so100_leader in its bash snippet, which re-introduced the same inconsistency the PR claims to fix. Replace all three with their so101 equivalents so the section is internally consistent with the paired Python API example. Refs huggingface#3219 Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Addresses the SO100 / SO101 naming inconsistency half of #3219.
Inside
docs/source/il_robots.mdx, bash commands and Python API examples previously mixed old SO100 class names with newer SO101 names. This PR makes the tutorial use SO101 consistently across every section where SO100/SO101 appear.Scope (explicit)
This PR only addresses the naming inconsistency. It does not fix the "rerun.io visualization not showing data" part of #3219 — that root cause appears unrelated to class naming and warrants separate investigation. Issue #3219 should stay open for the visualization problem after this lands.
Refs #3219
Changes
All SO100 → SO101 renames in
docs/source/il_robots.mdx:SO100Follower,SO100FollowerConfig,SO100Leader,SO100LeaderConfig→ SO101 equivalents.so100_follower/eval_so100/so100_leader→ SO101.The SO101 classes (
SO101Follower,SO101FollowerConfig,SO101Leader,SO101LeaderConfig) are already exported fromlerobot.robots.so_followerandlerobot.teleoperators.so_leader.Testing
Documentation-only change, no code paths affected. After the change:
```
$ grep -i so100 docs/source/il_robots.mdx
(no matches)
```
SO101 now appears uniformly where SO100 used to.
Notes
Keeping the PR as a draft for now — please convert to ready-for-review when you're happy with it.