From b88f1a01adc4728cb87ad7cffd905c0283832fb4 Mon Sep 17 00:00:00 2001 From: Rajat Ahuja Date: Mon, 3 Aug 2026 12:20:14 -0400 Subject: [PATCH] fix: comments --- src/crud/document.py | 4 ---- src/crud/representation.py | 4 +--- 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/src/crud/document.py b/src/crud/document.py index f808d004..36d9048d 100644 --- a/src/crud/document.py +++ b/src/crud/document.py @@ -355,10 +355,6 @@ async def query_documents( Returns: Sequence of matching documents """ - # A non-positive top_k means "no results wanted" -- return before embedding - # or querying. Turbopuffer rejects top_k=0 with a 400 (pgvector would - # silently do LIMIT 0), and callers derive top_k from budget arithmetic or - # LLM tool input, so neither is guaranteed positive. if top_k <= 0: return [] diff --git a/src/crud/representation.py b/src/crud/representation.py index 5807d475..57e6e8e1 100644 --- a/src/crud/representation.py +++ b/src/crud/representation.py @@ -319,9 +319,7 @@ class RepresentationManager: total = max_observations # Calculate how many observations to get from each source. - # Floor of 1 when a semantic query was explicitly requested: `total // 3` - # rounds to 0 for total < 3, which would allocate no budget to the source - # the caller actually asked to curate around. + # Floor of 1 when a semantic query was explicitly requested. semantic_observations = ( min( max(