search

johnmyleswhite / kNN.jl

The k-nearest neighbors algorithm in Julia
Other
22 stars 17 forks source link

example from docs doesn't work in julia 0.2.0 from the ubuntu ppa. #1

Closed heartpunk closed 10 years ago

heartpunk commented 10 years ago 
julia> predictions = knn(train_features, test_features, train_labels, k)
ERROR: distances not defined
 in knn at /home/vagrant/.julia/kNN/src/classify.jl:9
johnmyleswhite commented 10 years ago

I'll fix this tomorrow night. This package hasn't been looked at by me in a very long time.

heartpunk commented 10 years ago

Cool, thanks!

On Sun, Dec 22, 2013 at 9:46 PM, John Myles White notifications@github.comwrote:

I'll fix this tomorrow night. This package hasn't been looked at by me in a very long time.

— Reply to this email directly or view it on GitHubhttps://github.com/johnmyleswhite/kNN.jl/issues/1#issuecomment-31105260 .

johnmyleswhite commented 10 years ago

Still working on this. Kind of hate the current API, so it's going to be a massive redesign when released. Sorry for it taking a while.

johnmyleswhite commented 10 years ago

Closed by 319a5f019eea3524a2a8fbf5bf872565a044ebd2

There will be a lot more breaking changes in this package to come, but this new API is a start at something sane.

heartpunk commented 10 years ago

Thanks for fixing it, man! I appreciate it. :sushi: