From f60d1635889010f185f1dd30eef20377eef4e2ef Mon Sep 17 00:00:00 2001
From: Pepijn Kooijmans <pepijn@huggingface.co>
Date: Mon, 16 Mar 2026 02:58:28 +0100
Subject: [PATCH] refactor(leaderboard): use a single positional file arg for
 repo IDs

Replace --repo-ids and --repo-ids-file with a single positional
repo_ids_file argument. Lines starting with # are ignored.

Made-with: Cursor
---
 src/lerobot/scripts/lerobot_leaderboard.py | 35 ++++++----------------
 1 file changed, 9 insertions(+), 26 deletions(-)

diff --git a/src/lerobot/scripts/lerobot_leaderboard.py b/src/lerobot/scripts/lerobot_leaderboard.py
index de3208e43..0ef7a9697 100644
--- a/src/lerobot/scripts/lerobot_leaderboard.py
+++ b/src/lerobot/scripts/lerobot_leaderboard.py
@@ -21,14 +21,8 @@ sortable, filterable leaderboard table.
 
 Usage::
 
-    lerobot-leaderboard \
-        --repo-ids user/model_a,user/model_b \
-        --output leaderboard.html
-
-    # Or from a file listing repo IDs (one per line):
-    lerobot-leaderboard \
-        --repo-ids-file models.txt \
-        --output leaderboard.html
+    # models.txt contains one HF repo ID per line (lines starting with # are ignored)
+    lerobot-leaderboard models.txt --output leaderboard.html
 """
 
 from __future__ import annotations
@@ -548,15 +542,8 @@ def parse_args(argv: list[str] | None = None) -> argparse.Namespace:
         description="Generate an interactive eval leaderboard from Hub model repos.",
     )
     p.add_argument(
-        "--repo-ids",
+        "repo_ids_file",
         type=str,
-        default=None,
-        help="Comma-separated list of HF model repo IDs.",
-    )
-    p.add_argument(
-        "--repo-ids-file",
-        type=str,
-        default=None,
         help="Path to a text file with one repo ID per line.",
     )
     p.add_argument(
@@ -577,17 +564,13 @@ def parse_args(argv: list[str] | None = None) -> argparse.Namespace:
 def main(argv: list[str] | None = None):
     args = parse_args(argv)
 
-    repo_ids: list[str] = []
-    if args.repo_ids:
-        repo_ids.extend(r.strip() for r in args.repo_ids.split(",") if r.strip())
-    if args.repo_ids_file:
-        path = Path(args.repo_ids_file)
-        if not path.exists():
-            logger.error(f"File not found: {path}")
-            sys.exit(1)
-        repo_ids.extend(line.strip() for line in path.read_text().splitlines() if line.strip())
+    path = Path(args.repo_ids_file)
+    if not path.exists():
+        logger.error(f"File not found: {path}")
+        sys.exit(1)
+    repo_ids = [line.strip() for line in path.read_text().splitlines() if line.strip() and not line.startswith("#")]
     if not repo_ids:
-        logger.error("No repo IDs provided. Use --repo-ids or --repo-ids-file.")
+        logger.error(f"No repo IDs found in {path}")
         sys.exit(1)
 
     entries = fetch_all(repo_ids)