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

docs(configs): clarify the is_remote_target/is_remote split

Browse Source 
Add a comment explaining why JobConfig keeps both the staticmethod (tests
a raw target string from argv before a config exists) and the property
(accessor for an existing config instance).
This commit is contained in:
Nicolas Rabault
2026-06-24 11:30:06 +02:00
parent 8a3a411af6
commit 71a89d30f0
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/lerobot/configs/default.py
+3
View File
@@ -142,6 +142,9 @@ class JobConfig:
# Hub. A "lerobot" tag is always added; e.g. --job.tags '["lelab"]' adds more. # Hub. A "lerobot" tag is always added; e.g. --job.tags '["lelab"]' adds more.
tags: list[str] = field(default_factory=list) tags: list[str] = field(default_factory=list)
# Two entry points to the same predicate: the staticmethod tests a raw target string
# straight from argv (before any JobConfig exists, to decide dispatch early), while the
# property is the ergonomic accessor for code that already holds a config instance.
@staticmethod @staticmethod
def is_remote_target(target: str | None) -> bool: def is_remote_target(target: str | None) -> bool:
"""True when `target` names an HF Jobs flavor rather than a local run.""" """True when `target` names an HF Jobs flavor rather than a local run."""