Skip to content

Refactor make_knn_struct#40

Open
ahill187 wants to merge 136 commits intoShobiStassen:masterfrom
ahill187:refactor-knn-struct-2
Open

Refactor make_knn_struct#40
ahill187 wants to merge 136 commits intoShobiStassen:masterfrom
ahill187:refactor-knn-struct-2

Conversation

@ahill187
Copy link
Contributor

Description

In this PR:

  1. Added knn, ef_query, hnsw_param_ef_construction, hnsw_param_m, n_threads, x_data, distance_cluster as parameters to make_knn_struct
  2. Removed too_big and big_cluster
  3. Added docstrings
  4. Added tests
  5. Made sure ef_query was the right size

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant