Brummel/alpha-id
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

polish: drop redundant Filter import; document equal eval metrics

Browse Source
This commit is contained in:
Brummel
2026-05-18 19:29:50 +02:00
parent 6179d89931
commit 16fd91fa68
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/bin/alpha_id.rs
+2 -1
View File
@@ -37,7 +37,6 @@ enum Cmd {
fn run_eval(p: &Pipeline, m: Mode, cases: usize, seed: u64) -> (f64, f64) { 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::eval::{inject_errors, recall_at_k};
use alpha_id::model::Filter;
let billable: Vec<_> = p.entries.iter().enumerate() let billable: Vec<_> = p.entries.iter().enumerate()
.filter(|(_, e)| e.valid) .filter(|(_, e)| e.valid)
.filter(|(_, e)| alpha_id::corpus::primary_code(e) .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); sum += recall_at_k(&preds, &code, 10);
} }
let n = billable.len().max(1) as f64; 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) (sum / n, sum / n)
} }