[mlpack-git] [mlpack/mlpack] add cli executable for data_split (#650)

[Ryan Curtin]

I don't think you should rename the existing `TrainTestSplit()` to `LabelTrainTestSplit()`, you should just leave it named `TrainTestSplit()` and add additional overloads for when labels are not present.  To be honest, I think it would also be fine if we renamed the function the simpler `Split()`, but I am not picky about that part.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/650#issuecomment-221968720
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160526/98969437/attachment.html>