From f5174aaf2931513a48f7dea508e17d83618e762a Mon Sep 17 00:00:00 2001 From: sneak Date: Thu, 15 Oct 2026 17:19:47 +0000 Subject: [PATCH] Say half the CPUs plainly in DefaultMaxConcurrentProcessing The constant cpusPerImage gave a reason nothing else states, that each image uses two CPUs. The function now divides by 2, with the nolint comment the repo uses for such numbers. Model: opus-5-5 --- internal/config/config.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/internal/config/config.go b/internal/config/config.go index d91e3ea..1454863 100644 --- a/internal/config/config.go +++ b/internal/config/config.go @@ -42,9 +42,7 @@ const ( // (runtime.GOMAXPROCS(0), which follows a container's CPU limit), rounded // down, and at least 1. func DefaultMaxConcurrentProcessing() int { - const cpusPerImage = 2 - - return max(runtime.GOMAXPROCS(0)/cpusPerImage, 1) + return max(runtime.GOMAXPROCS(0)/2, 1) //nolint:mnd // half the CPUs } // Configuration key names.