From bdbf67ef9399a05e741c64c85e46e98fec69964e Mon Sep 17 00:00:00 2001 From: Ludovic Marie <136255825+ludovic111@users.noreply.github.com> Date: Mon, 26 Oct 2026 01:31:38 +0100 Subject: [PATCH] pr_ready: work out the affected harnesses once, not once per harness affected_here called parity.affected_harnesses inside the filter, once for each of the 343 harnesses. Since the selection reads the Rust items each probe reaches (#1964) one call takes about a minute and a half, so pr_ready sat for hours before its first step. Co-Authored-By: Claude Opus 5.5 --- tools/pr_ready.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/pr_ready.py b/tools/pr_ready.py index 505de28c1..4a3da6bae 100755 --- a/tools/pr_ready.py +++ b/tools/pr_ready.py @@ -164,7 +164,8 @@ def affected_here(changed): (parity.SHARED / "dependencies" / f"{name}.json").exists() for name in found) if not known and not changed & parity.SHARED_INPUTS: return None - return harness_names(name for name in found if name in parity.affected_harnesses(found, records, changed)) + affected = parity.affected_harnesses(found, records, changed) + return harness_names(name for name in found if name in affected) def short_list(names, limit=12):