admin/lerobot
1
0
Fork 0
mirror of https://github.com/huggingface/lerobot.git synced 2026-07-26 19:26:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(libero): inline init_states resolver behind single regex

Browse Source 
Collapse the three-style suffix stripper (split/re.sub/in) into one
compiled regex, drop the (Path, bool) tuple return, and move the
`_add_`/`_level` reshape branch into the caller so each branch loads
its own file and returns directly. Net: -11 lines, one fewer helper.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Pepijn
2026-04-14 18:39:09 +02:00
parent 394d5d4bb6
commit 5e80ae286e
1 changed files with 18 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/lerobot/envs/libero.py
+17 -37
View File
@@ -70,48 +70,28 @@ def _select_task_ids(total_tasks: int, task_ids: Iterable[int] | None) -> list[i
return ids return ids
def _resolve_libero_init_states_path(task: Any) -> tuple[Path, bool]: # LIBERO-plus perturbation variants encode the perturbation in the filename
"""Resolve the on-disk init_states path for a LIBERO task. # but on disk only the base `.pruned_init` exists — strip the suffix to match
# LIBERO-plus's own suite.get_task_init_states() (we reimplement it here so we
LIBERO-plus perturbation variants encode the perturbation in the filename # can pass weights_only=False for PyTorch 2.6+ numpy pickles).
(_table_N, _tb_N, _view_, _language_, _light_, _add_, _level) but only the _LIBERO_PERTURBATION_SUFFIX_RE = re.compile(r"_(?:language|view|light)_[^.]*|_(?:table|tb)_\d+")
base `.pruned_init` exists on disk. Mirror LIBERO-plus's own suffix-stripping
(libero/libero/benchmark/__init__.py) so we can load with `weights_only=False`
ourselves instead of delegating to the suite (whose `torch.load` call omits
that flag and fails on PyTorch 2.6+ numpy pickles).
Returns (path, needs_reshape) — `_add_`/`_level` variants store a flat
init_state that must be reshaped to (1, -1).
"""
filename = task.init_states_file
problem_folder = task.problem_folder
ext = filename.rsplit(".", 1)[-1]
root = Path(get_libero_path("init_states"))
needs_reshape = "_add_" in filename or "_level" in filename
if needs_reshape:
return root / "libero_newobj" / problem_folder / filename, True
if "_language_" in filename:
filename = filename.split("_language_")[0] + "." + ext
elif "_view_" in filename:
filename = filename.split("_view_")[0] + "." + ext
else:
if "_table_" in filename:
filename = re.sub(r"_table_\d+", "", filename)
if "_tb_" in filename:
filename = re.sub(r"_tb_\d+", "", filename)
if "_light_" in filename:
filename = filename.split("_light_")[0] + "." + ext
return root / problem_folder / filename, False
def get_task_init_states(task_suite: Any, i: int) -> np.ndarray: def get_task_init_states(task_suite: Any, i: int) -> np.ndarray:
task = task_suite.tasks[i] task = task_suite.tasks[i]
init_states_path, needs_reshape = _resolve_libero_init_states_path(task) filename = Path(task.init_states_file)
root = Path(get_libero_path("init_states"))
# `_add_` / `_level` variants store extra-object layouts under libero_newobj/
# as a flat array that must be reshaped to (1, -1).
if "_add_" in filename.name or "_level" in filename.name:
init_states_path = root / "libero_newobj" / task.problem_folder / filename.name
init_states = torch.load(init_states_path, weights_only=False) # nosec B614 init_states = torch.load(init_states_path, weights_only=False) # nosec B614
if needs_reshape: return init_states.reshape(1, -1)
init_states = init_states.reshape(1, -1)
return init_states stripped = _LIBERO_PERTURBATION_SUFFIX_RE.sub("", filename.stem) + filename.suffix
init_states_path = root / task.problem_folder / stripped
return torch.load(init_states_path, weights_only=False) # nosec B614
def get_libero_dummy_action(): def get_libero_dummy_action():