dkm icon indicating copy to clipboard operation
dkm copied to clipboard

Implement N-dimensional array class to replace `std::vector<std::array<T, N>>`

Open genbattle opened this issue 5 years ago • 2 comments

Issues (#8 and #11) have been raised about the flexibility of the current std::vector<std::array<T, N>> pattern used for passing data matrices in and out of this library.

A more flexible alternative is to have a dedicated N-dimensional array class (dkm::ndarray<T>) which has convenient constructors that take both the existing std::vector<std::array<T, N>> and std::vector<std::vector<T>>, which has been requested for some use cases. This class would also have methods to explicitly convert back to these data types after running the kmeans algorithm.

genbattle avatar Mar 03 '19 23:03 genbattle

Thanks for the great library! fwiw, I created Ruby bindings for it but want to wait until there's a way to specify dimensions at runtime before releasing it.

ankane avatar Dec 21 '19 18:12 ankane

Hey, thanks for your interest in this library.

This is the next improvement on my TODO list, so as soon as I have some time I'll start refactoring towards this goal.

genbattle avatar Dec 25 '19 10:12 genbattle