From 6aaf931c9ba0d9746640daa17347a57d8dbad718 Mon Sep 17 00:00:00 2001 From: teernisse Date: Fri, 6 Mar 2026 16:14:49 -0500 Subject: [PATCH] fix(embedding): guard is_multiple_of() progress logs against zero is_multiple_of(N) returns true for 0, which caused debug/info progress messages to fire at doc_num=0 (the start of every page) rather than only at the intended 50/100 milestones. Add != 0 check to both the debug (every 50) and info (every 100) log sites. Co-Authored-By: Claude Opus 4.6 --- src/embedding/pipeline.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/embedding/pipeline.rs b/src/embedding/pipeline.rs index 3c166fc..5aaa18a 100644 --- a/src/embedding/pipeline.rs +++ b/src/embedding/pipeline.rs @@ -161,7 +161,7 @@ async fn embed_page( continue; } - if page_normal_docs.is_multiple_of(50) { + if page_normal_docs != 0 && page_normal_docs.is_multiple_of(50) { debug!( doc_id = doc.document_id, doc_num = page_normal_docs, @@ -169,7 +169,7 @@ async fn embed_page( "Chunking document" ); } - if page_normal_docs.is_multiple_of(100) { + if page_normal_docs != 0 && page_normal_docs.is_multiple_of(100) { info!( doc_id = doc.document_id, content_bytes = doc.content_text.len(),