<p>In <a href="https://github.com/mlpack/mlpack/pull/457#discussion_r40792244">src/mlpack/methods/softmax_regression/softmax_regression.hpp</a>:</p>
<pre style='color:#555'>> @@ -141,13 +140,8 @@ class SoftmaxRegression
> * @param numClasses Number of classes for classification.
> * @return Objective value of the final point.
> */
> - double Train(const arma::mat &data, const arma::vec& labels,
> - const size_t numClasses);
> -
> - //! Sets the size of the input vector.
> - size_t& InputSize() { return inputSize; }
> - //! Gets the size of the input vector.
> - size_t InputSize() const { return inputSize; }
</pre>
<p>Sure, I'll go ahead and merge this and change <code>InputSize()</code> to <code>FeatureSize()</code>. It should be <code>n_rows</code>, not <code>n_cols</code> -- Armadillo matrices are column-major, so each column corresponds to an observation and each row corresponds to a feature.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/mlpack/mlpack/pull/457/files#r40792244">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFPOzenxd2U6LjPZ3e3aIjjgJUWKIks5o29pYgaJpZM4GFbCy.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/mlpack/mlpack/pull/457/files#r40792244"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>