revert as original was correct

harp-intel · harp-intel · commit a0888b3648f7 · 2025-11-21T11:06:04.000-08:00
Signed-off-by: Harper, Jason M &lt;jason.m.harper@intel.com&gt;
diff --git a/cmd/metrics/loader_util.go b/cmd/metrics/loader_util.go
@@ -166,7 +166,7 @@ func replaceConstants(metrics []MetricDefinition, metadata Metadata) ([]MetricDe
 		return nil, fmt.Errorf("unknown granularity: %s", flagGranularity)
 	}
 	coresPerSocket := fmt.Sprintf("%f", float64(metadata.CoresPerSocket))
-	chasPerSocket := fmt.Sprintf("%f", float64(len(metadata.UncoreDeviceIDs["cha"]))/float64(metadata.SocketCount))
+	chasPerSocket := fmt.Sprintf("%f", float64(len(metadata.UncoreDeviceIDs["cha"])))
 	socketCount := fmt.Sprintf("%f", float64(metadata.SocketCount))
 	hyperThreadingOn := fmt.Sprintf("%t", metadata.ThreadsPerCore > 1)
 	threadsPerCore := fmt.Sprintf("%f", float64(metadata.ThreadsPerCore))

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,7 @@ func replaceConstants(metrics []MetricDefinition, metadata Metadata) ([]MetricDe`
`166`	`166`	`return nil, fmt.Errorf("unknown granularity: %s", flagGranularity)`
`167`	`167`	`}`
`168`	`168`	`coresPerSocket := fmt.Sprintf("%f", float64(metadata.CoresPerSocket))`
`169`		`- chasPerSocket := fmt.Sprintf("%f", float64(len(metadata.UncoreDeviceIDs["cha"]))/float64(metadata.SocketCount))`
	`169`	`+ chasPerSocket := fmt.Sprintf("%f", float64(len(metadata.UncoreDeviceIDs["cha"])))`
`170`	`170`	`socketCount := fmt.Sprintf("%f", float64(metadata.SocketCount))`
`171`	`171`	`hyperThreadingOn := fmt.Sprintf("%t", metadata.ThreadsPerCore > 1)`
`172`	`172`	`threadsPerCore := fmt.Sprintf("%f", float64(metadata.ThreadsPerCore))`