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.

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

[FEA] Expose pylibraft function to compute the kmeans inertia #972

Closed
cjnolet opened this issue Nov 1, 2022 · 0 comments · Fixed by #1028
Closed

[FEA] Expose pylibraft function to compute the kmeans inertia #972

cjnolet opened this issue Nov 1, 2022 · 0 comments · Fixed by #1028
Assignees
@benfred
Labels
feature request New feature or request

Comments

@cjnolet
Copy link
Member

cjnolet commented Nov 1, 2022

This function is already exposed on RAFT's public APIs and just needs to be wrapped w/ pylibraft so it can be used in Python. This is in support of the Scikit-learn plugins PoC.
@cjnolet cjnolet added the feature request New feature or request label Nov 1, 2022
@benfred benfred self-assigned this Nov 1, 2022
benfred added a commit to benfred/raft that referenced this issue Nov 17, 2022
@benfred
Expose cluster_cost to python
403e955 
Add cython bindings for the cluster_cost function, to allow
computing inertia from python.

Closes rapidsai#972
@benfred benfred mentioned this issue Nov 17, 2022
Merged
rapids-bot bot pushed a commit that referenced this issue Nov 21, 2022
@benfred
Expose cluster_cost to python (#1028)
1eff0a1 
Add cython bindings for the cluster_cost function, to allow computing inertia from python.

Closes #972

Authors:
  - Ben Frederickson (https://github.com/benfred)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: #1028
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benfred benfred

Labels
feature request New feature or request
Projects
VS/ML/DM Primitives Release Board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Expose cluster_cost to python benfred/raft
2 participants
@benfred @cjnolet