[mlpack-git] [mlpack] implement command line programs of softmaxRegression (#466)
Ryan Curtin
notifications at github.com
Thu Oct 29 08:42:28 EDT 2015
Great, this is very useful and I'd love to add it in. A couple of things:
* Do you mind if I change it to `softmax_regression_main.cpp`?
* I think it would be useful to have support for loading and saving models; it looks like you have support for saving models via `--output_file`, but we should allow the user to load one too. Take a look at the interface I implemented for logistic regression; we can basically imitate that interface: https://github.com/mlpack/mlpack/blob/master/src/mlpack/methods/logistic_regression/logistic_regression_main.cpp
* Can you open another issue for the problem you're having loading CSVs on Windows?
* I can add the documentation under the `PROGRAM_INFO()` macro, so don't worry about that unless you want to.
* If you want, it might be worth it to add an `--optimizer` option, just like in `logistic_regression`, because the `SoftmaxRegression` class can use different optimizers. Your call (if you don't do it, I will, it's pretty easy).
* I'd avoid using `auto` for clarity's sake.
Thanks for taking the time to write this!
---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/466#issuecomment-152166872
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20151029/4742fa65/attachment.html>
More information about the mlpack-git
mailing list