mirror of
https://github.com/kevinveenbirkenbach/computer-playbook.git
synced 2026-11-06 02:54:26 +00:00
test(perf): raise the cold domains render budget to 100s
make quality runs test-external, test-integration, test-lint and test-unit concurrently, each with its own xdist pool, so a wall-clock assertion in one of them measures machine contention as much as code. The cold domains render takes 7.34s isolated on this host and took 91.50s inside that gate - a factor of 12.5 with no code between the two measurements. Profiling the cold render puts copy.deepcopy on top by a wide margin (4.35M calls, 3.4s tottime), but those copies are contractual rather than accidental: utils/cache/README.md:36 states that cache getters hand out shared instances and that every mutating caller must deep-copy first. Removing them means replacing that invariant with copy-on-write or immutable structures, which is a redesign of the cache contract and not a fix for a flaky budget. Only the cold-start ceiling moves. The assertions that catch a real cache regression are untouched: warm calls still have to stay under WARM_CALLS_BUDGET_SECONDS = 0.5, and the cache still has to be at least 50x faster than the cold render. The sibling applications budget is left at 60s deliberately; it measures 7.61s isolated and is exposed the same way, so it is the next candidate if the gate flakes there. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
1 parent
435ff42219
commit
19d56f07e4
1 file changed
+2
-2
@@ -244,8 +244,8 @@ class TestRuntimeLookupPerformance(unittest.TestCase):
|
||||
|
||||
self.assertLess(
|
||||
warm_elapsed,
|
||||
60.0,
|
||||
f"Cold domains render took {warm_elapsed:.2f}s (budget 60s).",
|
||||
100.0,
|
||||
f"Cold domains render took {warm_elapsed:.2f}s (budget 100s).",
|
||||
)
|
||||
|
||||
|
||||
|
||||
Reference in new issue
Block a user