-
Notifications
You must be signed in to change notification settings - Fork 62
refactor(robot): unify the robot stack - one sim process shape, batched-first bridge/agent, grouped evals #481
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
lukass16
wants to merge
40
commits into
main
Choose a base branch
from
lukass/phys-experimental
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
3966c31
wip: save robot agent/recorder/bridge state before vectorized taskset…
lukass16 ebd1cbe
refactor(robot): serve every sim with one process shape (SimThread)
lukass16 0321435
refactor(telemetry): extract shared robot recording into hud.telemetr…
lukass16 ed4cb29
refactor(robot): batched-first RobotBridge with slots grading; add Gy…
lukass16 37826e4
feat(robot): declarative gym envs (env.gym) and hud.wrap trace streaming
lukass16 0f84f9c
refactor(agents): unify RobotAgent to drive N>=1 env slots
lukass16 8b27cae
feat(eval): grouped rollouts via rollout_group and Taskset.run(num_en…
lukass16 9e222ff
docs(robots): update for the unified robot stack
lukass16 47264de
feat(agents): size BatchedModel to the live concurrency by default
lukass16 4d05aae
refactor(eval): rename rollout_group to vec_rollout
lukass16 d4f7efa
fix(agents): satisfy strict pyright in the robot agent stack
lukass16 2fe6bfd
fix(agents): make DatasetWriter work with derived contracts
lukass16 923e546
fix(agents): stop recording inference for finished env slots
lukass16 7710838
docs(robots): run parameters table; batching and vec_rollout updates
lukass16 a622ba5
feat(environment): serve concurrent control sessions
jdchawla29 e353e12
merge: bring in concurrent control sessions (a622ba5)
lukass16 0faf857
refactor(agents): drop per-agent max_steps override on RobotAgent
lukass16 6e4a811
refactor(robot): spawn every sim as its own fork-free process; declar…
lukass16 748ed4c
feat(eval): add Run.started carrying the full tasks.start reply
lukass16 b5258cd
feat(environment/robot): lock RobotEndpoint; reset/result carry a token
lukass16 fded695
feat(environment/robot): slot registry + barrier stepping in the bridge
lukass16 2c698d4
feat(capabilities/robot): RobotClient.connect claims a slot by token
lukass16 278376f
refactor(agents/robot): drop drive() for a single scalar rollout loop
lukass16 880b902
refactor(eval): drop vec_rollout/num_envs; add Shared provider
lukass16 aef1196
docs(robots): document sessions-based vectorized evals and Shared
lukass16 121d945
docs(agents): tighten BatchedModel and BatchedAgent docstrings
lukass16 8b42d5e
feat(agents): share one LeRobot dataset across concurrent rollouts
lukass16 b521d96
docs(robot): tighten RobotClient and Environment.gym docstrings
lukass16 42f6e28
refactor(environment/robot): fold SimThread into bridge; move GymBrid…
lukass16 d1d7e23
refactor(hud): drop lazy hud.wrap export from package root
lukass16 1df6832
docs(robot): point wrap at hud.environment.robot, not hud.wrap
lukass16 08d424e
docs(environment/robot): shorten the bridge module docstring
lukass16 0a2d2cf
fix(environment/robot): barrier over all claimed slots, not just conn…
lukass16 9884ccc
docs(agents/robot): clarify max_steps is a call-site kwarg with defau…
lukass16 9f0cc5a
fix(robot): fail fast when the slot claim is missing instead of deadl…
lukass16 5ae0641
fix(environment/robot): reject mismatched claim kwargs; fix hold shap…
lukass16 9e1df70
fix(agents/robot): honor subclass max_steps and skip acting when alre…
lukass16 083122f
fix(robot): barrier, claim, telemetry, and gym-argv robustness from r…
lukass16 e878d30
feat(robot): make the slot token optional for single-env bridges
lukass16 cc47fff
feat(environment/robot): lazy Isaac spawn — serve the wire before the…
lukass16 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -60,6 +60,7 @@ | |
| "instrument", | ||
| ] | ||
|
|
||
|
|
||
| try: | ||
| from .version import __version__ | ||
| except ImportError: | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.