From f78f39c6e5eae47fdb7fcea060aa2931a60a5e2b Mon Sep 17 00:00:00 2001
From: Brendan O'Leary <brendan@olearycrew.com>
Date: Tue, 10 Mar 2026 00:02:55 -0400
Subject: [PATCH] Randomize model-to-instance assignment

Shuffles the model list before distributing across instances (round-robin).
This prevents systematic biases like alphabetically-first models always
landing on instance 0.

Use --no-shuffle to restore deterministic (alphabetical) assignment if needed.
---
 orchestrate_vultr.py | 20 +++++++++++++++++---
 1 file changed, 17 insertions(+), 3 deletions(-)

diff --git a/orchestrate_vultr.py b/orchestrate_vultr.py
index 99a69e4..0b70f33 100755
--- a/orchestrate_vultr.py
+++ b/orchestrate_vultr.py
@@ -31,6 +31,7 @@
 import argparse
 import json
 import os
+import random
 import socket
 import subprocess
 import sys
@@ -227,16 +228,24 @@ def launch_instance(
     return label, instance_id, ip
 
 
-def distribute_models(models: list[str], count: int) -> list[list[str]]:
+def distribute_models(models: list[str], count: int, *, shuffle: bool = True) -> list[list[str]]:
     """
     Distribute models across N instances using round-robin assignment.
 
+    When shuffle=True (default), the model order is randomized before distribution.
+    This prevents the same models from always landing on the same instance positions,
+    which helps avoid systematic biases in benchmarking (e.g., first instances getting
+    all the "A" models alphabetically).
+
     Examples:
       9 models, 3 instances → [[m0,m3,m6], [m1,m4,m7], [m2,m5,m8]]
       3 models, 5 instances → [[m0], [m1], [m2], [], []]
     """
+    working_models = list(models)
+    if shuffle:
+        random.shuffle(working_models)
     buckets: list[list[str]] = [[] for _ in range(count)]
-    for i, model in enumerate(models):
+    for i, model in enumerate(working_models):
         buckets[i % count].append(model)
     return buckets
 
@@ -319,6 +328,11 @@ def main() -> int:
         action="store_true",
         help="Enable verbose debug logging for scheduling/executor flow",
     )
+    parser.add_argument(
+        "--no-shuffle",
+        action="store_true",
+        help="Disable randomization of model order (default: models are shuffled before distribution)",
+    )
 
     args = parser.parse_args()
 
@@ -340,7 +354,7 @@ def main() -> int:
         ssh_keys=args.ssh_keys,
     )
 
-    buckets = distribute_models(models, args.count)
+    buckets = distribute_models(models, args.count, shuffle=not args.no_shuffle)
     non_empty = [(i, b) for i, b in enumerate(buckets) if b]
 
     if args.debug: