-
Notifications
You must be signed in to change notification settings - Fork 43
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
Support pre-assigned the centriods #136
Comments
KMedoids does not support array in init for now, you are welcome to PR for it. Mainly you have to add the code in _initialize_medoids, add docstring and also add a test to verify that it works. I'm looking forward to your PR. |
@TimotheeMathieu |
You have to make some pytest compatible tests (look at examples in the existing tests). Then, to run the tests locally, execute |
Here is the PR |
I think this is not working atm: |
are you sure that you installed the github version ? This is not released yet, you have to install the git version of scikit-learn-extra for this to work. |
You're right. It works after installing with |
@TimotheeMathieu Do you have a PyPI account ? I should add you as a maintainer there, so you can publish a new release if needed. |
Yes, my account is |
I sent you the PyPI invite, thanks for your work @TimotheeMathieu ! Well, there are a few PRs and ~300 repos that depend on this package so there is some interest. Sorry myself I stepped down from scikit-learn maintenance including this repo to move to other projects. But if you have a chance you should talk with people at the scikit-learn foundation, to see how to best move forward, particularly since most people involved are at INRIA in France also. I'll follow up on this by email. |
KMeans from sci-kit learn
init
parameters allows array. Does KMedoids also support it? I can submit pull requests for itIf an array is passed, it should be of shape (n_clusters, n_features) and gives the initial centers.
The text was updated successfully, but these errors were encountered: