fix: comments
This commit is contained in:
parent
70f4dc5f48
commit
b88f1a01ad
|
|
@ -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 []
|
||||
|
||||
|
|
|
|||
|
|
@ -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(
|
||||
|
|
|
|||
Loading…
Reference in New Issue