<p>The reason I put it in the <code>Impute()</code> function itself is because it is only useful for the case where we are using only one matrix (i.e. input matrix = output matrix).  When we are using two matrices, we need to scan the whole matrix anyway, so <code>dirtyDimensions</code> won't help us there.  I think it's a little odd to offer an API that allows a user to call <code>Impute()</code> with an input and output matrix, or to call <code>Impute()</code> with a matrix and a list of dirty dimensions.  I think the list of dirty dimensions shouldn't be required as an input parameter, since the imputer already has a <code>DatasetMapper</code> object it can use to calculate the dirty dimensions.  Hopefully what I've written makes sense... let me know if I can clarify.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/694#issuecomment-234614033">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJ4bFJAQ8LjK-QK_VSVjb8vgx7v7YtDZks5qYQYigaJpZM4I07W-">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFESShp4xjvd7PuCHdl7JiH-NXp9aks5qYQYigaJpZM4I07W-.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/694#issuecomment-234614033"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>