From 4cf9091fb4eaeb9b85943051e22e674d27187cfe Mon Sep 17 00:00:00 2001 From: TianyeDong Date: Sun, 5 Jul 2026 12:36:32 -0400 Subject: [PATCH] refactor(rlix-validation): drop unreachable per-engine contiguity loop in C6-mapping The preceding check already asserts infer_device_mapping == range(len), so every per_engine slice is contiguous by construction and the loop can never raise. Remove the dead code. Co-Authored-By: Claude Opus 4.8 --- miles/utils/rlix_validation.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/miles/utils/rlix_validation.py b/miles/utils/rlix_validation.py index 3d582e67c7..fcdb6da619 100644 --- a/miles/utils/rlix_validation.py +++ b/miles/utils/rlix_validation.py @@ -330,14 +330,6 @@ def assert_rlix_topology(args: Any, sglang_config: Any | None = None) -> None: f"infer_device_mapping starting at 0 (got {infer_device_mapping}); " f"non-contiguous / custom ordering is a follow-up adapter (F12 / A18)." ) - for engine_index, start in enumerate(range(0, len(infer_device_mapping), per_engine)): - group = infer_device_mapping[start : start + per_engine] - expected = list(range(group[0], group[0] + per_engine)) - if group != expected: - raise RuntimeError( - f"C6-mapping: infer engine {engine_index} must occupy contiguous " - f"GPUs in first build; got {group}, expected {expected}" - ) # --- C17: RLix mode disables RadixTreeMiddleware (partial_rollout + radix_tree forbidden) middleware_paths = getattr(args, "miles_router_middleware_paths", None) or []