Skip to content

Refactor knngraph_full -> create_knn_graph#15

Merged
ahill187 merged 6 commits intomasterfrom
create-knn-graph
Aug 13, 2024
Merged

Refactor knngraph_full -> create_knn_graph#15
ahill187 merged 6 commits intomasterfrom
create-knn-graph

Conversation

@ahill187
Copy link
Owner

Description

In this PR:

  1. Renamed knngraph_full -> create_knn_graph
  2. Cleaned up the function
  3. Added tests
  4. Added docstrings

See ShobiStassen#36.

@ahill187 ahill187 self-assigned this Aug 13, 2024
@ahill187 ahill187 added the refactor Code refactoring label Aug 13, 2024
@ahill187 ahill187 merged commit 65434d8 into master Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant