Merge pull request #6 from tkuchiki/fix-percentile

Fix percentile calculation
tkuchiki · Jul 26, 2022 · ffb7e56 · ffb7e56
2 parents 96df080 + a9a6e32
commit ffb7e56
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/stats/printer.go b/stats/printer.go
@@ -189,7 +189,7 @@ func NewPrinter(w io.Writer, val, format string, percentiles []int, printOptions
 		p.keywords = helper.SplitCSV(val)
 		for _, key := range p.keywords {
 			p.headers = append(p.headers, p.headersMap[key])
-			if key == "all" {
+			if key == string(allMode) {
 				p.keywords = keywords(percentiles, outputKeywords[allMode])
 				p.headers = defaultHeaders(percentiles, outputKeywords[allMode])
 				p.all = true

diff --git a/stats/query.go b/stats/query.go
@@ -493,8 +493,8 @@ func percentRank(n int, pi int) int {
 
 	p := float64(pi) / 100.0
 	pos := int(float64(n+1) * p)
-	if pos < 0 {
-		pos = 0
+	if pos <= 0 {
+		return 0
 	}
 
 	return pos - 1