From 8fe20ba678346558b5dffe003157f9dc4d87690f Mon Sep 17 00:00:00 2001 From: Michael Chus Date: Mon, 6 Apr 2026 22:30:59 +0300 Subject: [PATCH] Fix benchmark scoring: PowerSustain uses default power limit PowerSustainScore now uses DefaultPowerLimitW as reference so a manually reduced power limit does not inflate the score. Falls back to enforced limit if default is unavailable. Co-Authored-By: Claude Sonnet 4.6 --- audit/internal/platform/benchmark.go | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/audit/internal/platform/benchmark.go b/audit/internal/platform/benchmark.go index 9533315..814ec25 100644 --- a/audit/internal/platform/benchmark.go +++ b/audit/internal/platform/benchmark.go @@ -705,8 +705,14 @@ func scoreBenchmarkGPUResult(gpu BenchmarkGPUResult) BenchmarkScorecard { score.ComputeScore += precision.TeraOpsPerSec } } - if gpu.PowerLimitW > 0 { - score.PowerSustainScore = math.Min(100, (gpu.Steady.AvgPowerW/gpu.PowerLimitW)*100) + // Use default power limit for sustain score so a manually reduced limit + // does not inflate the score. Fall back to enforced limit if default unknown. + referencePowerW := gpu.DefaultPowerLimitW + if referencePowerW <= 0 { + referencePowerW = gpu.PowerLimitW + } + if referencePowerW > 0 { + score.PowerSustainScore = math.Min(100, (gpu.Steady.AvgPowerW/referencePowerW)*100) } runtimeUS := math.Max(1, gpu.Steady.DurationSec*1e6) thermalRatio := float64(gpu.Throttle.HWThermalSlowdownUS+gpu.Throttle.SWThermalSlowdownUS) / runtimeUS