Skip to content

refactor: hnswlib performance #1727

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JoanFM merged 15 commits into from
Jul 31, 2023
Merged

refactor: hnswlib performance #1727

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9aebf2e
refactor: hnswlib performance
Jul 26, 2023
02849c4
Merge branch 'main' of https://github.com/docarray/docarray into refa…
Jul 26, 2023
3f56d44
fix: reconstruct on filter
Jul 27, 2023
f8bb5f9
fix: reconstruct from proper type
Jul 27, 2023
ae33fe6
fix: get the proper embedding back
Jul 27, 2023
a91e23e
chore: avoid extra debugging (#1730)
Jul 27, 2023
fa9b9ce
Merge branch 'main' of https://github.com/docarray/docarray into refa…
Jul 27, 2023
79b0fea
Merge branch 'get-current-count-hnswlib' of https://github.com/docarr…
Jul 27, 2023
58ee168
chore: add comment to take over later
Jul 28, 2023
83e55f5
test: refactor hnswlib test subindex
Jul 28, 2023
281cf7e
Merge branch 'test-refactor' of https://github.com/docarray/docarray …
Jul 28, 2023
2ea5804
chore: fix test
Jul 28, 2023
1c8337e
Merge branch 'main' of https://github.com/docarray/docarray into refa…
Jul 28, 2023
8c0bf6c
chore: apply optim
Jul 28, 2023
c062c8c
Merge branch 'main' into refactor-hnswlib-performance
Jul 30, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
chore: fix test
  • Loading branch information
Joan Fontanals Martinez committed Jul 28, 2023
commit 2ea5804a26a1f94157025b9754357adfdf59b058
13 changes: 0 additions & 13 deletions docarray/index/backends/hnswlib.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -579,18 +579,6 @@ def _doc_from_bytes(
data
) # I cannot reconstruct directly the DA object because it may fail at validation because embedding may not be Optional
for k, v in reconstruct_embeddings.items():
print(f' k {k}')
# access = k.split('__')
# proto = pb
# python_schema = self.out_schema
# for field in access:
# print(f' field {field} and {python_schema} and {proto}')
# proto = proto.data[field]
# python_schema = python_schema._get_field_type(field)
# node_proto = python_schema._docarray_from_ndarray(np.array(v))._to_node_protobuf()
# proto.MergeFrom(node_proto)

print(f'v {v}')
node_proto = (
self.out_schema.__fields__[k]
.type_._docarray_from_ndarray(np.array(v))
Expand All @@ -599,7 +587,6 @@ def _doc_from_bytes(
pb.data[k].MergeFrom(node_proto)

doc = schema_cls.from_protobuf(pb)
print(f'doc {doc}')
return doc

def _get_root_doc_id(self, id: str, root: str, sub: str) -> str:
Expand Down
6 changes: 0 additions & 6 deletions tests/index/elastic/v7/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,3 @@ services:
- ES_JAVA_OPTS=-Xmx1024m
ports:
- "9200:9200"
networks:
- elastic

networks:
elastic:
name: elastic
6 changes: 0 additions & 6 deletions tests/index/elastic/v8/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,3 @@ services:
- ES_JAVA_OPTS=-Xmx1024m
ports:
- "9200:9200"
networks:
- elastic

networks:
elastic:
name: elastic