Skip to content

Commit 3ffb276

Browse files
roypatbchalios
authored andcommitted
fix: re-add missing parenthesis to A/B script output
Fixes: fc6a36e Signed-off-by: Patrick Roy <[email protected]>
1 parent 0506126 commit 3ffb276

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tools/ab_test.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -285,7 +285,7 @@ def ab_performance_test(
285285
msg = (
286286
f"\033[0;32m[Firecracker A/B-Test Runner]\033[0m A/B-testing shows a change of "
287287
f"{format_with_reduced_unit(result.statistic, unit)}, or {result.statistic / old_mean:.2%}, "
288-
f"(from {format_with_reduced_unit(old_mean, unit)} to {format_with_reduced_unit(new_mean, unit)}"
288+
f"(from {format_with_reduced_unit(old_mean, unit)} to {format_with_reduced_unit(new_mean, unit)}) "
289289
f"for metric \033[1m{metric}\033[0m with \033[0;31m\033[1mp={result.pvalue}\033[0m. "
290290
f"This means that observing a change of this magnitude or worse, assuming that performance "
291291
f"characteristics did not change across the tested commits, has a probability of {result.pvalue:.2%}. "

0 commit comments

Comments
 (0)