genbattle / dkm

A generic C++11 k-means clustering implementation
MIT License
209 stars 47 forks source link

readme wrong information for return type of kmeans_lloyd() #15

Closed UriahHuang closed 4 years ago

UriahHuang commented 5 years ago

in section Usage of readme, the return type of kmeans_lloyd() should be std::tuple<std::vector<std::array<T, N>>, std::vector> not std::tuple<std::array<T, N>, std::vector> which make me confuse at the first look

genbattle commented 5 years ago

Thanks for raising this, I'll fix this next time I push some code up.

kam3k commented 4 years ago

I found this error before seeing that this issue was already brought up and submitted a PR: https://github.com/genbattle/dkm/pull/16

genbattle commented 4 years ago

PR merged. Fixed.