Repository navigation
feat(scripts): add the Objaverse importer and SmolVLA training entry - #4
Merged
Merged
Conversation
scripts/README.md already listed both. recorder.py imported lerobot without declaring it, so the new extra covers recording and training. Co-authored-by: Cursor <cursoragent@cursor.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
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.
Problem and outcome
scripts/README.mdlistsscripts/assets/import_objaverse.pyandscripts/train_smolvla.py, but neither file was in the tree. The eight Objaverse assets inassets/objects/had no reproducible source, and there was no training entry for the recorded LeRobot datasets.Both scripts are now in the tree, unchanged from the versions that produced the assets and the post-training runs.
pyproject.tomlgains alerobotextra;robotsmith/sim/recorder.pyalready importedlerobotlazily without declaring it.import_objaverse.pyneedsobjaverseand RocRecon'smesh_to_urdf(it raises with install instructions if RocRecon is missing).train_smolvla.pyneedstorchandlerobot(pip install -e ".[sim,lerobot]").Layer
Evidence
Neither script was run end to end for this PR: the import needs network access to Objaverse plus RocRecon, and training needs a GPU.
AI use
main, added the extra and changelog lines, and wrote this description.lerobotextra and whether either script needs a smoke run before merge.