From 2632be20344c4371e03c836c7a085680166cc58f Mon Sep 17 00:00:00 2001
From: andrews05 <andrew@digerati.co.nz>
Date: Fri, 31 Jan 2025 06:28:42 +1300
Subject: [PATCH] Fix size passed to fast filter evaluator (#672)

The evaluator uses the `estimated_output_size` internally but we were
passing only the idat size in the fast filter evaluator. At level `o2`,
or higher levels when using `--fast`, this could cause a slight bias
towards the None and Bigrams filters for indexed images. The actual
impact to results is incredibly minor but it's definitely incorrect so
we should fix it.
---
 src/lib.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib.rs b/src/lib.rs
index 0832f08d..c6775f9f 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -509,7 +509,7 @@ fn perform_trials(
                 opts.optimize_alpha,
             );
             if let Some(result) = &eval_result {
-                eval.set_best_size(result.idat_data.len());
+                eval.set_best_size(result.estimated_output_size());
             }
             eval.try_image(image.clone());
             if let Some(result) = eval.get_best_candidate() {