From cba72b753dccf34ecd969cf3c27ac6db333a40a1 Mon Sep 17 00:00:00 2001 From: Felix Weiglhofer Date: Tue, 20 Jan 2026 15:05:24 +0100 Subject: [PATCH 1/2] GPU: Don't override --recoSteps flags in standalone. --- .../Standalone/Benchmark/standalone.cxx | 21 +++++++++++-------- 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx b/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx index 5fa9da23d7423..014a01291f1e6 100644 --- a/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx +++ b/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx @@ -413,6 +413,18 @@ int32_t SetupReconstruction() steps.steps.setBits(gpudatatypes::RecoStep::TPCClusterFinding, false); } + // Set settings for synchronous + GPUChainTracking::ApplySyncSettings(procSet, recSet, steps.steps, configStandalone.testSyncAsync || configStandalone.testSync, configStandalone.rundEdx); + int32_t runAsyncQA = procSet.runQA && !configStandalone.testSyncAsyncQcInSync ? procSet.runQA : 0; + if (configStandalone.testSyncAsync) { + procSet.eventDisplay = nullptr; + if (!configStandalone.testSyncAsyncQcInSync) { + procSet.runQA = false; + } + } + + // Apply --recoSteps flag last so it takes precedence + // E.g. ApplySyncSettings might enable TPCdEdx, but might not be needed if only clusterizer was requested if (configStandalone.recoSteps >= 0) { steps.steps &= configStandalone.recoSteps; } @@ -432,15 +444,6 @@ int32_t SetupReconstruction() } } - // Set settings for synchronous - GPUChainTracking::ApplySyncSettings(procSet, recSet, steps.steps, configStandalone.testSyncAsync || configStandalone.testSync, configStandalone.rundEdx); - int32_t runAsyncQA = procSet.runQA && !configStandalone.testSyncAsyncQcInSync ? procSet.runQA : 0; - if (configStandalone.testSyncAsync) { - procSet.eventDisplay = nullptr; - if (!configStandalone.testSyncAsyncQcInSync) { - procSet.runQA = false; - } - } rec->SetSettings(&grp, &recSet, &procSet, &steps); if (configStandalone.proc.doublePipeline) { From 138bf68fbc5d41205d67f08efb503cfc4b43fadd Mon Sep 17 00:00:00 2001 From: ALICE Action Bot Date: Tue, 20 Jan 2026 15:08:59 +0000 Subject: [PATCH 2/2] Please consider the following formatting changes --- GPU/GPUTracking/Standalone/Benchmark/standalone.cxx | 1 - 1 file changed, 1 deletion(-) diff --git a/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx b/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx index 014a01291f1e6..a2e74c45fcb86 100644 --- a/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx +++ b/GPU/GPUTracking/Standalone/Benchmark/standalone.cxx @@ -444,7 +444,6 @@ int32_t SetupReconstruction() } } - rec->SetSettings(&grp, &recSet, &procSet, &steps); if (configStandalone.proc.doublePipeline) { recPipeline->SetSettings(&grp, &recSet, &procSet, &steps);