Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Nearest centroids #5

Merged
merged 2 commits into from Mar 28, 2012

Conversation

Projects
None yet
2 participants

Fixes for the flaws I pointed out in comments to your PR, as well some minor stuff.

larsmans added some commits Mar 27, 2012

DOC small fixes to NearestCentroid classifier
Point out connection to Rocchio classifier from NLP/IR.
BUG disallow shrinking with sparse data in NearestCentroid
+ various small fixes.

Shrinking would implicitly densify the input matrix, rendering it harmful
on large datasets.

robertlayton added a commit that referenced this pull request Mar 28, 2012

@robertlayton robertlayton merged commit 46fcc12 into robertlayton:nearest_centroids Mar 28, 2012

Owner

robertlayton commented Mar 28, 2012

Thanks for doing this! (I had a comment, but removed it after reading your message in the other thread.)

robertlayton pushed a commit that referenced this pull request Jan 19, 2014

Merged pull request #5 from larsmans/master.
Some more copyediting, as well as a small section on cross-validation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment