Skip to content

fix(rl): preserve teleop intervention flag in step_env_and_process_transition#3273

Open
remete618 wants to merge 1 commit intohuggingface:mainfrom
remete618:fix/teleop-intervention-flag-overwrite
Open

fix(rl): preserve teleop intervention flag in step_env_and_process_transition#3273
remete618 wants to merge 1 commit intohuggingface:mainfrom
remete618:fix/teleop-intervention-flag-overwrite

Conversation

@remete618
Copy link
Copy Markdown

Summary

What changed

Two lines in src/lerobot/rl/gym_manipulator.py (lines 554-555):

# Before (env info overwrites processor info)
new_info = processed_action_transition[TransitionKey.INFO].copy()
new_info.update(info)

# After (processor info overwrites env info)
new_info = info.copy()
new_info.update(processed_action_transition[TransitionKey.INFO])

Test plan

  • Verify TeleopEvents.IS_INTERVENTION is True in transition info when teleop processor sets it
  • Existing tests pass

@s1lent4gnt s1lent4gnt self-assigned this Apr 8, 2026
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.

gym_manipulator: Teleop intervention flag gets overwritten to False in step_env_and_process_transition

2 participants