Skip to content

Commit

Permalink
fix: format iters/s warnings properly
Browse files Browse the repository at this point in the history
Signed-off-by: Jérôme Benoit <[email protected]>
  • Loading branch information
jerome-benoit committed Oct 4, 2024
1 parent 2d5675f commit 175d7f6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/reporter/terminal/table.js
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ export function warning(
benchmark.stats.throughput.rmoe > highRelativeMarginOfError
) {
warnings.push(
`${bold(colors, yellow(colors, 'Warning'))}: ${bold(colors, cyan(colors, benchmark.name))} has a high latency throughput margin of error: ${red(colors, errorMargin(benchmark.stats.throughput.rmoe))}`
`${bold(colors, yellow(colors, 'Warning'))}: ${bold(colors, cyan(colors, benchmark.name))} has a high latency throughput margin of error: ${red(colors, itersPerSecond(benchmark.stats.throughput.rmoe))}`
)
}
if (latency && benchmark.stats.latency.mad > 0) {
Expand All @@ -178,7 +178,7 @@ export function warning(
}
if (throughput && benchmark.stats.throughput.mad > 0) {
warnings.push(
`${bold(colors, yellow(colors, 'Warning'))}: ${bold(colors, cyan(colors, benchmark.name))} has a non zero throughput median absolute deviation: ${red(colors, duration(benchmark.stats.throughput.mad))}`
`${bold(colors, yellow(colors, 'Warning'))}: ${bold(colors, cyan(colors, benchmark.name))} has a non zero throughput median absolute deviation: ${red(colors, itersPerSecond(benchmark.stats.throughput.mad))}`
)
}
}
Expand Down

0 comments on commit 175d7f6

Please sign in to comment.