Skip to content

Commit a41a91d

Browse files
committed
fix(bench): print p999 in latency stdout for operator visibility
The harness already computes metrics.p999_ns; surfacing it during manual runs costs nothing and matches the legacy bench_latency output. BMF wire-up for latency_p999_ns / latency_max_ns is deferred to PR 6 alongside threshold configuration.
1 parent d3b8d10 commit a41a91d

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

benchmarks/nim/bench_latency.nim

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,9 +133,13 @@ proc runVariant(
133133
)
134134
else:
135135
raise newException(ValueError, "unknown variant: " & variant)
136+
# p999 / max are printed for operator visibility but NOT emitted to BMF
137+
# at this PR (latency_p999_ns / latency_max_ns wire-up is deferred to
138+
# PR 6 along with their threshold configuration).
136139
echo fmt" p50: {metrics.p50_ns:.0f} ns"
137140
echo fmt" p95: {metrics.p95_ns:.0f} ns"
138141
echo fmt" p99: {metrics.p99_ns:.0f} ns"
142+
echo fmt" p999: {metrics.p999_ns:.0f} ns"
139143
echo fmt" max: {metrics.max_ns:.0f} ns"
140144
echo fmt" samples: {metrics.samples}"
141145
echo ""

0 commit comments

Comments
 (0)