From 1fdc2b3aa3858a5a7f64b8193a3b3bc9d7bd3644 Mon Sep 17 00:00:00 2001 From: Jonathan Shook Date: Sat, 9 Sep 2023 16:03:43 -0500 Subject: [PATCH] fix operator precedence bug in F1 computation --- .../engine/extensions/computefunctions/ComputeFunctions.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/engine-extensions/src/main/java/io/nosqlbench/engine/extensions/computefunctions/ComputeFunctions.java b/engine-extensions/src/main/java/io/nosqlbench/engine/extensions/computefunctions/ComputeFunctions.java index 09d38358d..c64da6686 100644 --- a/engine-extensions/src/main/java/io/nosqlbench/engine/extensions/computefunctions/ComputeFunctions.java +++ b/engine-extensions/src/main/java/io/nosqlbench/engine/extensions/computefunctions/ComputeFunctions.java @@ -139,7 +139,7 @@ public class ComputeFunctions { public static double F1(int[] relevant, int[] actual, int k) { double recallAtK = recall(relevant, actual, k); double precisionAtK = precision(relevant, actual, k); - return 2.0d * ((recallAtK * precisionAtK) / (recallAtK + precisionAtK)); + return (2.0d * (recallAtK * precisionAtK)) / (recallAtK + precisionAtK); } public static double F1(long[] relevant, long[] actual) { @@ -149,7 +149,7 @@ public class ComputeFunctions { public static double F1(long[] relevant, long[] actual, int k) { double recallAtK = recall(relevant, actual, k); double precisionAtK = precision(relevant, actual, k); - return 2.0d * ((recallAtK * precisionAtK) / (recallAtK + precisionAtK)); + return (2.0d * (recallAtK * precisionAtK)) / (recallAtK + precisionAtK); } /**