Skip to content

Commit cba72b7

Browse files
committed
GPU: Don't override --recoSteps flags in standalone.
1 parent 9ca7f3a commit cba72b7

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

GPU/GPUTracking/Standalone/Benchmark/standalone.cxx

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -413,6 +413,18 @@ int32_t SetupReconstruction()
413413
steps.steps.setBits(gpudatatypes::RecoStep::TPCClusterFinding, false);
414414
}
415415

416+
// Set settings for synchronous
417+
GPUChainTracking::ApplySyncSettings(procSet, recSet, steps.steps, configStandalone.testSyncAsync || configStandalone.testSync, configStandalone.rundEdx);
418+
int32_t runAsyncQA = procSet.runQA && !configStandalone.testSyncAsyncQcInSync ? procSet.runQA : 0;
419+
if (configStandalone.testSyncAsync) {
420+
procSet.eventDisplay = nullptr;
421+
if (!configStandalone.testSyncAsyncQcInSync) {
422+
procSet.runQA = false;
423+
}
424+
}
425+
426+
// Apply --recoSteps flag last so it takes precedence
427+
// E.g. ApplySyncSettings might enable TPCdEdx, but might not be needed if only clusterizer was requested
416428
if (configStandalone.recoSteps >= 0) {
417429
steps.steps &= configStandalone.recoSteps;
418430
}
@@ -432,15 +444,6 @@ int32_t SetupReconstruction()
432444
}
433445
}
434446

435-
// Set settings for synchronous
436-
GPUChainTracking::ApplySyncSettings(procSet, recSet, steps.steps, configStandalone.testSyncAsync || configStandalone.testSync, configStandalone.rundEdx);
437-
int32_t runAsyncQA = procSet.runQA && !configStandalone.testSyncAsyncQcInSync ? procSet.runQA : 0;
438-
if (configStandalone.testSyncAsync) {
439-
procSet.eventDisplay = nullptr;
440-
if (!configStandalone.testSyncAsyncQcInSync) {
441-
procSet.runQA = false;
442-
}
443-
}
444447

445448
rec->SetSettings(&grp, &recSet, &procSet, &steps);
446449
if (configStandalone.proc.doublePipeline) {

0 commit comments

Comments
 (0)