From f37be3edbee60f3a09a5183788b91eb19f0c07d1 Mon Sep 17 00:00:00 2001 From: Steven Palma Date: Tue, 28 Jul 2026 18:41:28 +0200 Subject: [PATCH] fix(eval): prevent eval_policy crash when start_seed is None and num_envs>1 (#4203) * fix(eval): align seed list length with num_envs when unseeded eval_policy appended a single None per batch to all_seeds on the unseeded path while the reward and success lists grew by num_envs. The per-episode zip(..., strict=True) then raised ValueError for num_envs > 1. Extend all_seeds by num_envs so the lists stay aligned. * chore(tests): delete lerobot_eval test --------- Co-authored-by: Devin Lai --- src/lerobot/scripts/lerobot_eval.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lerobot/scripts/lerobot_eval.py b/src/lerobot/scripts/lerobot_eval.py index bcab78594..de4936653 100644 --- a/src/lerobot/scripts/lerobot_eval.py +++ b/src/lerobot/scripts/lerobot_eval.py @@ -564,7 +564,7 @@ def eval_policy( if seeds: all_seeds.extend(seeds) else: - all_seeds.append(None) + all_seeds.extend([None] * env.num_envs) # FIXME: episode_data is either None or it doesn't exist if return_episode_data: