Skip to content

refactor: remove K-medoids algorithm completely - #45

Merged
Harsh16gupta merged 1 commit into
masterfrom
removing-k-medoids-completely
Aug 15, 2026
Merged

refactor: remove K-medoids algorithm completely#45
Harsh16gupta merged 1 commit into
masterfrom
removing-k-medoids-completely

Conversation

@Harsh16gupta

Copy link
Copy Markdown
Collaborator

As discussed with my mentor, we have decided to remove K-medoids completely from the repository. K-medoids is signficantly slower than K-Means and HDBSCAN on vector spaces and is no longer used in default categorization strategies.

Changes

  • Deleted src/pipeline/clustering/kmedoids.ts and unit test test/pipeline/clustering/kmedoids.test.ts.
  • Removed kmedoids algorithm variant from ClusteringAlgorithm type in src/types/cluster.ts.
  • Refactored autoK.ts and benchmark.ts to remove kmedoids selection, imports, and conditions.
  • Updated settings display string in SettingsPage.tsx.
  • Updated test cases in autoK.test.ts.

@Harsh16gupta Harsh16gupta self-assigned this Aug 14, 2026
@Harsh16gupta
Harsh16gupta force-pushed the removing-k-medoids-completely branch from a0c9e4c to 22df07d Compare August 15, 2026 11:10
@Harsh16gupta

Copy link
Copy Markdown
Collaborator Author

The PR just removes the k-medoids code completely from the codebase, so I am merging it.

@Harsh16gupta
Harsh16gupta merged commit c414bee into master Aug 15, 2026
1 check passed
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

Successfully merging this pull request may close these issues.

1 participant