minor adjustments

Author: john-b-yang

codeclash/tournaments/ladder.py

@@ -195,6 +195,16 @@ def __init__(
         self.config = config
         self.ladder = config["ladder"]
         self.player = config["player"]
+        # CC:Ladder semantics REQUIRE push: True. Cross-rung carry-over (each rung continues from the
+        # previous rung's codebase) is delivered by checking out the pushed branch; with push: False
+        # every rung silently restarts from the starter template (no carry-over) and --resume is
+        # impossible. Reject it loudly rather than produce a degenerate run.
+        if not self.player.get("push"):
+            raise ValueError(
+                "CC:Ladder requires the player's `push: True`. With `push: False`, per-rung carry-over "
+                "silently degrades to restarting every rung from the starter template (no hill-climb), "
+                "and --resume cannot work. Set `push: True` in the player config."
+            )
         self.rounds = config["tournament"]["rounds"]
         self.sims = config["game"]["sims_per_round"]
         self.min_round_wins, self.win_last_k = resolve_ladder_rules(config.get("ladder_rules", {}), self.rounds)

configs/ladder/ladder_rules.yaml

@@ -1,8 +1,8 @@
 min_round_wins: 2
 win_last_k: 0
-# Optional: stop a rung as soon as the climber has won `min_round_wins` rounds instead of playing
-# them all. Requires win_last_k == 0. Off by default. Uncomment to enable:
-# early_clinch: true
+# Stop a rung as soon as the climber has won `min_round_wins` rounds instead of playing them all.
+# Requires win_last_k == 0. Saves redundant post-clinch rounds; same "highest rung reached" metric.
+early_clinch: true
 # Optional: skip playing a rung whose carried-over bot already dominates round 0 (faster eval,
 # same "highest rung reached" metric). Off by default. Uncomment to enable:
 # fast_forward: