-
Notifications
You must be signed in to change notification settings - Fork 189
feat: Add k-means clustering #27
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.
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
Conversation
HairlessVillager
commented
Apr 8, 2024
Hey, gay! Could you please add commas at the end of lines? There are so many errors!
Hey, gay! Could you please add commas at the end of lines? There are so many errors!
Hi gay! Where are the errors? Could you share a screenshot of them?
HairlessVillager
commented
Apr 8, 2024
I'm sorry, actually it's ;.
I don't think they are errors, and semicolons are optional. But it's good to add them to be consistent with the style in this repo. You could fork and add them and then make a PR. 🥇
HairlessVillager
commented
Apr 8, 2024
Let's forget it. There is no big problem even if it is not changed. I'm feeling lazy now.😴
Hi there!
I'd like to add a new algorithm
k-means clusteringto this repo, is it appropriate to merge? Please help review, thanks in advance!The new visualizer needs an extra Scatter Chart, so there are relevant PRs to other repos, please help review them, too.
image