<blockquote>
<p>So I think that each column of the output matrix should correspond to a maximal input</p>
</blockquote>

<p>The purpose of this function is visualize the learning results of autoencoder , even it is non-image data, you could use this function to visualize it too(ex : audio data). Extract the maximal inputs into each col is more general, but  I can not imagine what is the used of it, maybe this is a little bit over designed.</p>

<blockquote>
<p>Also, I think that there should be a bool parameter to MaximalInputs() to determine whether or not the output is going to be rescaled (I'd default it to false, because not all applications are image applications).</p>
</blockquote>

<p>I agree with you, not all of the applications are image applications, but the purpose of this function is visualize the results learned by autoencoder, so I think it should scale the range by default, because most of the image readers would assume the range of the images lie between [0,255] or [0, 1]</p>

<p>Appreciate your inputs, this kind of conversations and code reviews really help me learn something.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/mlpack/mlpack/pull/465#issuecomment-152437189">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFJ0qlJNxo2baDGdj_WpO-D--5kpPks5pAwCCgaJpZM4GWT_f.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/465#issuecomment-152437189"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>