theirongolddev/gitlore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(embedding): guard is_multiple_of() progress logs against zero

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
teernisse
2026-03-06 16:14:49 -05:00
parent af167e2086
commit 6aaf931c9b
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/embedding/pipeline.rs
View File

@@ -161,7 +161,7 @@ async fn embed_page(
continue; continue;
} }
if page_normal_docs.is_multiple_of(50) { if page_normal_docs != 0 && page_normal_docs.is_multiple_of(50) {
debug!( debug!(
doc_id = doc.document_id, doc_id = doc.document_id,
doc_num = page_normal_docs, doc_num = page_normal_docs,
@@ -169,7 +169,7 @@ async fn embed_page(
"Chunking document" "Chunking document"
); );
} }
if page_normal_docs.is_multiple_of(100) { if page_normal_docs != 0 && page_normal_docs.is_multiple_of(100) {
info!( info!(
doc_id = doc.document_id, doc_id = doc.document_id,
content_bytes = doc.content_text.len(), content_bytes = doc.content_text.len(),