From 16fd91fa685c1dca2f6e7826f544d65dc5c2cb02 Mon Sep 17 00:00:00 2001 From: Brummel Date: Mon, 18 May 2026 19:29:50 +0200 Subject: [PATCH] polish: drop redundant Filter import; document equal eval metrics --- src/bin/alpha_id.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/bin/alpha_id.rs b/src/bin/alpha_id.rs index af2cb0a..26e3130 100644 --- a/src/bin/alpha_id.rs +++ b/src/bin/alpha_id.rs @@ -37,7 +37,6 @@ enum Cmd { fn run_eval(p: &Pipeline, m: Mode, cases: usize, seed: u64) -> (f64, f64) { use alpha_id::eval::{inject_errors, recall_at_k}; - use alpha_id::model::Filter; let billable: Vec<_> = p.entries.iter().enumerate() .filter(|(_, e)| e.valid) .filter(|(_, e)| alpha_id::corpus::primary_code(e) @@ -53,6 +52,8 @@ fn run_eval(p: &Pipeline, m: Mode, cases: usize, seed: u64) -> (f64, f64) { sum += recall_at_k(&preds, &code, 10); } let n = billable.len().max(1) as f64; + // Both metrics are equal by construction: the loop runs only over + // billable cases, so Recall@10 == BillableRecall@10 here. (sum / n, sum / n) }