Skip to content
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

Jump to bottom

Constructing the KNN graph should allow a sparse data matrix #39

Closed
pmacg opened this issue Jan 6, 2022 · 0 comments · Fixed by #40
Closed

Constructing the KNN graph should allow a sparse data matrix #39

pmacg opened this issue Jan 6, 2022 · 0 comments · Fixed by #40
Assignees
@pmacg
Labels
bug Something isn't working

Comments

@pmacg
Copy link
Owner

pmacg commented Jan 6, 2022

Currently, when passing a sparse data matrix to the knn graph method, an error is thrown. This shouldn't happen.
@pmacg pmacg added the bug Something isn't working label Jan 6, 2022
@pmacg pmacg self-assigned this Jan 6, 2022
pmacg added a commit that referenced this issue Jan 6, 2022
@pmacg
Issue #39 - knn construction with sparse data matrix
b6f801c
@pmacg pmacg mentioned this issue Jan 6, 2022
Merged
pmacg added a commit that referenced this issue Jan 6, 2022
@pmacg
Issue #39 - typo in test script
2a61041
@pmacg pmacg closed this as completed in #40 Jan 6, 2022
pmacg added a commit that referenced this issue Jan 6, 2022
@pmacg
Merge pull request #40 from pmacg/iss39-sparse-data-matrix
dc8cbb0 
Issue #39 - KNN construction with sparse data matrix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pmacg pmacg

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #39 - KNN construction with sparse data matrix pmacg/py-sgtl
1 participant
@pmacg