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.

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

[KMeans default algorithm is being set to Lloyd] #69

Open
SadeghKrmi opened this issue Apr 15, 2023 · 0 comments
Open

[KMeans default algorithm is being set to Lloyd] #69

SadeghKrmi opened this issue Apr 15, 2023 · 0 comments

Comments

@SadeghKrmi
Copy link

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 scikit-learn/scikit-learn@bacc91c

Describe alternatives you've considered
change default algorithm to "LIoyd", reference to scikit-learn/scikit-learn@bacc91c

Additional context
change default algorithm to "LIoyd", reference to scikit-learn/scikit-learn@bacc91c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant