From aae68e34487b63aa1fe1d7d876316f8aa6d63863 Mon Sep 17 00:00:00 2001 From: Pepijn Kooijmans Date: Mon, 23 Mar 2026 18:57:44 +0100 Subject: [PATCH] fix(docker): use recursive glob for deeply nested asset zip structure The LIBERO-plus assets.zip has a deeply nested path (inspire/hdd/project/.../assets) that didn't match the shallow glob. Use recursive glob to find assets/scenes regardless of nesting depth. Made-with: Cursor --- docker/Dockerfile.benchmark | 3 ++- docker/Dockerfile.eval-libero-plus | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/docker/Dockerfile.benchmark b/docker/Dockerfile.benchmark index 7c05c9e1f..75df1f810 100644 --- a/docker/Dockerfile.benchmark +++ b/docker/Dockerfile.benchmark @@ -133,7 +133,8 @@ assets_dst = os.path.join(get_libero_path('benchmark_root'), 'assets'); \ print(f'Downloading LIBERO-plus assets to {assets_dst}...'); \ zp = hf_hub_download('Sylvest/LIBERO-plus', 'assets.zip', repo_type='dataset', local_dir='/tmp/lp-dl'); \ zipfile.ZipFile(zp).extractall('/tmp/lp-unzip'); \ -src = glob.glob('/tmp/lp-unzip/inspire/*/assets')[0]; \ +hits = glob.glob('/tmp/lp-unzip/**/assets/scenes', recursive=True); \ +src = os.path.dirname(hits[0]); \ shutil.move(src, assets_dst); \ shutil.rmtree('/tmp/lp-dl', True); shutil.rmtree('/tmp/lp-unzip', True); \ print(f'Assets installed: {os.listdir(assets_dst)[:5]}...')" ;; \ diff --git a/docker/Dockerfile.eval-libero-plus b/docker/Dockerfile.eval-libero-plus index 80e38b8bb..4f312468e 100644 --- a/docker/Dockerfile.eval-libero-plus +++ b/docker/Dockerfile.eval-libero-plus @@ -51,7 +51,8 @@ assets_dst = os.path.join(get_libero_path('benchmark_root'), 'assets'); \ print(f'Downloading LIBERO-plus assets to {assets_dst}...'); \ zp = hf_hub_download('Sylvest/LIBERO-plus', 'assets.zip', repo_type='dataset', local_dir='/tmp/lp-dl'); \ zipfile.ZipFile(zp).extractall('/tmp/lp-unzip'); \ -src = glob.glob('/tmp/lp-unzip/inspire/*/assets')[0]; \ +hits = glob.glob('/tmp/lp-unzip/**/assets/scenes', recursive=True); \ +src = os.path.dirname(hits[0]); \ shutil.move(src, assets_dst); \ shutil.rmtree('/tmp/lp-dl', True); shutil.rmtree('/tmp/lp-unzip', True); \ print(f'Assets installed: {os.listdir(assets_dst)[:5]}...')"