ageron / ageron/handson-ml3

[KMeans default algorithm is being set to Lloyd]

Open
#69 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
14.1k
Forks
5.3k
PR merge metrics
No merged PRs in 30d

Description

Thanks for helping us improve this project!

**Before you create this issue**
Please make sure you are using the latest updated code and libraries: see https://github.com/ageron/handson-ml3/blob/main/INSTALL.md#update-this-project-and-its-libraries

Also please make sure to read the FAQ (https://github.com/ageron/handson-ml3#faq) and search for existing issues (both open and closed), as your question may already have been answered: https://github.com/ageron/handson-ml3/issues

**Is your feature request related to a problem? Please describe.**
Notebook *09_unsupervised_learning.ipynb* section "Accelerated K-Means"

_For Elkan's variant of K-Means, use algorithm="elkan". For regular KMeans, use algorithm="full". The default is "auto", which uses the full algorithm since Scikit-Learn 1.1 (it used Elkan's algorithm before that)._

**Describe the solution you'd like**
change default algorithm to "LIoyd", reference to https://github.com/scikit-learn/scikit-learn/commit/bacc91cf1d4531bcc91aa60893fdf7df319485ec

**Describe alternatives you've considered**
change default algorithm to "LIoyd", reference to https://github.com/scikit-learn/scikit-learn/commit/bacc91cf1d4531bcc91aa60893fdf7df319485ec

**Additional context**
change default algorithm to "LIoyd", reference to https://github.com/scikit-learn/scikit-learn/commit/bacc91cf1d4531bcc91aa60893fdf7df319485ec

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.