Skip to content

Commit af52fed

Browse files
authored
fix(cli): Fix filter condition for chunk inclusion (#287)
1 parent 410a67a commit af52fed

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

src/vectorcode/subcommands/query/__init__.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ async def get_query_result_files(
9393
num_query = configs.n_result
9494
if QueryInclude.chunk in configs.include:
9595
if filter:
96-
filter = {"$and": [filter.copy(), {"$gte": 0}]}
96+
filter = {"$and": [filter.copy(), {"start": {"$gte": 0}}]}
9797
else:
9898
filter["start"] = {"$gte": 0}
9999
else:
@@ -122,9 +122,8 @@ async def get_query_result_files(
122122
return []
123123

124124
reranker = get_reranker(configs)
125-
return await reranker.rerank(
126-
convert_query_results(chroma_query_results, configs.query)
127-
)
125+
converted_results = convert_query_results(chroma_query_results, configs.query)
126+
return await reranker.rerank(converted_results)
128127

129128

130129
async def build_query_results(

tests/subcommands/query/test_query.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ async def test_get_query_result_chunks_with_query_exclude(mock_collection, mock_
238238
mock_collection.query.assert_called_once()
239239
_, kwargs = mock_collection.query.call_args
240240
assert kwargs["where"] == {
241-
"$and": [{"path": {"$nin": ["/excluded/path.py"]}}, {"$gte": 0}]
241+
"$and": [{"path": {"$nin": ["/excluded/path.py"]}}, {"start": {"$gte": 0}}]
242242
}
243243

244244

0 commit comments

Comments
 (0)