Passing number of centroids unused
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 35/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- rust
- Domain
- machine-learning
Research direction
Open src/koans/06_update/02_centroids_array2.rs and inspect compute_centroids, focusing on whether n_centroids is needed for the intended centroid calculation. Compare the function signature with the surrounding koan and determine whether the argument should affect the result or be removed; the issue is done when that behavior is resolved consistently.
Written by the indexing model from the issue text.
Description
In update koan there is
pub fn compute_centroids(
n_centroids: usize,
// (n_observations, n_features)
observations: &ArrayBase<impl Data<Elem = f64>, Ix2>,
// (n_observations,)
cluster_memberships: &ArrayBase<impl Data<Elem = usize>, Ix1>,
) -> Array2<f64> {
but here the n_centroids is not being used for anything in the solution, and thus it doesn't seem necessary. Maybe there is a bug here?
- Dominant language
- Jupyter Notebook
- Stars
- 116
- Forks
- 20
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from LukeMathWalker/ndarray-koans
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
All issues in LukeMathWalker/ndarray-koans
Similar issues
-
bug module:classification
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
low AI-generated score register-submission
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
UKGovernmentBEIS/inspect_evals#2521 · 2 comments ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
stanfordnlp/stanza#1683 · 2 comments ·
-
wer_score / mer_score / cer_score return 0 (a perfect lower-is-better score) for an empty prediction Open
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
stanford-crfm/helm#4359 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
ml-explore/mlx-lm#1915 · 1 comment ·