Skip to content

refactor: do not recompute every time num_docs #1729

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 3 commits into from
Jul 26, 2023
Merged

refactor: do not recompute every time num_docs #1729

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f95b623
refactor: do not recompute every time num_docs
Jul 26, 2023
3ca5780
fix: k more than num docs
jupyterjazz Jul 26, 2023
dbdc990
Merge branch 'main' of https://github.com/docarray/docarray into fix-…
Jul 26, 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
fix: k more than num docs 
Signed-off-by: jupyterjazz <[email protected]>
  • Loading branch information
@jupyterjazz
jupyterjazz committed Jul 26, 2023
commit 3ca5780b9f5c89769e8ead133475151d205105d3
2 changes: 1 addition & 1 deletion docarray/index/backends/hnswlib.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -608,7 +608,7 @@ def _search_and_filter(
return _FindResultBatched(documents=[], scores=[]) # type: ignore

# Set limit as the minimum of the provided limit and the total number of documents
limit = limit
limit = min(limit, self.num_docs())

# Ensure the search field is in the HNSW indices
if search_field not in self._hnsw_indices:
Expand Down