Skip to content

Clustering #14

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
Merged
merged 18 commits into from
Nov 5, 2024
Merged

Clustering #14

merged 18 commits into from
Nov 5, 2024

Conversation

@jideoyelayo1
Copy link
Owner

@jideoyelayo1 jideoyelayo1 commented Nov 4, 2024 •
edited by trimmedjay
Loading

  • Hierarchical Clustering (Agglomerative)
  • K means
  • KNN

@jideoyelayo1 jideoyelayo1 added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 4, 2024
@jideoyelayo1 jideoyelayo1 self-assigned this Nov 4, 2024
@jideoyelayo1 jideoyelayo1 linked an issue Nov 4, 2024 that may be closed by this pull request
K-Nearest_Neighbours #8
Closed
@jideoyelayo1 jideoyelayo1 linked an issue Nov 4, 2024 that may be closed by this pull request
Clustering is bad #16
Closed
@jideoyelayo1 jideoyelayo1 linked an issue Nov 4, 2024 that may be closed by this pull request
Add more test cases to all #17
Closed
trimmedjay
trimmedjay approved these changes Nov 5, 2024
View reviewed changes
Copy link
Collaborator

@trimmedjay trimmedjay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It passed the tests so its good with me

jlasses
jlasses approved these changes Nov 5, 2024
View reviewed changes
Copy link
Collaborator

@jlasses jlasses left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like we should make the branch changes smaller. So we can see more progression. @jideoyelayo1 @trimmedjay

@jideoyelayo1 jideoyelayo1 merged commit 20068d3 into main Nov 5, 2024
16 checks passed
@jideoyelayo1 jideoyelayo1 deleted the clustering branch November 5, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jlasses jlasses jlasses approved these changes

@trimmedjay trimmedjay Awaiting requested review from trimmedjay

Assignees

@jideoyelayo1 jideoyelayo1

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add more test cases to all Clustering is bad K-Nearest_Neighbours

4 participants

@jideoyelayo1 @trimmedjay @jlasses