From 2972d235a3f64178c3f88bc8f8a9ace5bc73d4ea Mon Sep 17 00:00:00 2001 From: Ed_ Date: Wed, 10 Jun 2026 20:50:55 -0400 Subject: [PATCH] fix(io_pool): correct stale docstring (4 threads -> 8 threads) Per IO_POOL_MAX_WORKERS = 8 (set in commit 4a338486 on 2026-06-06 to relieve contention during batched sims), the pool actually has 8 workers, not 4. The docstring was stale. Also added the SHAs of the 4->8 bump for traceability. --- src/io_pool.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/io_pool.py b/src/io_pool.py index ac46afa8..2d794498 100644 --- a/src/io_pool.py +++ b/src/io_pool.py @@ -24,9 +24,11 @@ IO_POOL_THREAD_NAME_PREFIX: str = "controller-io" def make_io_pool(max_workers: int = IO_POOL_MAX_WORKERS) -> ThreadPoolExecutor: """Create the shared AppController I/O pool. - 4 worker threads, named "controller-io-N". Used for warmup, log pruning, - disk-bound subsystem init, and any other background work that should - not spin up its own thread. + 8 worker threads (per `IO_POOL_MAX_WORKERS = 8` above; bumped from 4 + in commit 4a338486 on 2026-06-06 to relieve contention during batched + sims), named "controller-io-N". Used for warmup, log pruning, disk- + bound subsystem init, and any other background work that should not + spin up its own thread. Caller is responsible for shutdown (e.g. controller.shutdown()). """