Skip to content

Commit 8fa0994

Browse files
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
1 parent ff87da3 commit 8fa0994

File tree

5 files changed

+19
-9
lines changed

5 files changed

+19
-9
lines changed

engine/clients/chroma/configure.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,10 @@ def clean(self):
3535

3636
def recreate(self, dataset: Dataset, collection_params):
3737
params = self.collection_params
38-
params["metadata"] = dict({"hnsw:space": self.DISTANCE_MAPPING.get(dataset.config.distance)}, **params.get('metadata', {}))
38+
params["metadata"] = dict(
39+
{"hnsw:space": self.DISTANCE_MAPPING.get(dataset.config.distance)},
40+
**params.get("metadata", {}),
41+
)
3942
self.client.create_collection(
4043
name=CHROMA_COLLECTION_NAME,
4144
**params,

engine/clients/chroma/parser.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,9 @@
99

1010
class ChromaConditionParser(BaseConditionParser):
1111
def build_condition(
12-
self, and_subfilters: Optional[List[Where]], or_subfilters: Optional[List[Where]]
12+
self,
13+
and_subfilters: Optional[List[Where]],
14+
or_subfilters: Optional[List[Where]],
1315
) -> Where:
1416
condition: Where = {}
1517
if and_subfilters is not None:
@@ -44,7 +46,9 @@ def build_range_filter(
4446
"$lte": lte,
4547
"$gte": gte,
4648
}
47-
filters: OperatorExpression = {k: v for k, v in raw_filters.items() if v is not None}
49+
filters: OperatorExpression = {
50+
k: v for k, v in raw_filters.items() if v is not None
51+
}
4852
return {field_name: filters}
4953

5054
def build_geo_filter(

engine/clients/chroma/search.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
from typing import List, Tuple
22

3-
from chromadb import HttpClient, ClientAPI, Settings
3+
from chromadb import ClientAPI, HttpClient, Settings
44
from chromadb.api.types import IncludeEnum
55

66
from dataset_reader.base_reader import Query
@@ -32,10 +32,13 @@ def search_one(cls, query: Query, top: int) -> List[Tuple[int, float]]:
3232
include=[IncludeEnum.distances],
3333
)
3434

35-
return [(int(hit[0]), float(hit[1])) for hit in zip(res["ids"][0], res["distances"][0])]
35+
return [
36+
(int(hit[0]), float(hit[1]))
37+
for hit in zip(res["ids"][0], res["distances"][0])
38+
]
3639

3740
def setup_search(self):
3841
metadata = self.collection.metadata.copy()
39-
metadata.pop("hnsw:space", None) # Not allowed in the collection.modify method
40-
metadata.update(self.search_params.get('metadata', {}))
42+
metadata.pop("hnsw:space", None) # Not allowed in the collection.modify method
43+
metadata.update(self.search_params.get("metadata", {}))
4144
self.collection.modify(metadata=metadata)

engine/clients/chroma/upload.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
from typing import List
22

3-
from chromadb import HttpClient, ClientAPI, Settings
3+
from chromadb import ClientAPI, HttpClient, Settings
44

55
from dataset_reader.base_reader import Record
66
from engine.base_client.upload import BaseUploader

experiments/configurations/chroma-single-node.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,5 +23,5 @@ See https://cookbook.chromadb.dev/core/configuration/#hnsw-configuration
2323
non-default not in use.
2424

2525
"parallel": 16 # implemented in base_client
26-
"batch_size": 1024 # implemented in base_client
26+
"batch_size": 1024 # implemented in base_client
2727

0 commit comments

Comments
 (0)