<p>Sorry for the slow response here.  I hope to have a chance to look soon, but I can't promise anything with paper deadlines coming up.  I think maybe we can expand on the tests a little bit; could you consider corner cases?  For example, what happens if a user passes an invalid number of dimensions or invalid data to the reduce() function?</p>

<p>Another useful test to add might be to take a dataset and pad it with many dimensions which are all zero.  In this case, you should be able to reduce it to the original number of dimensions while preserving the exact dissimilarity matrix.</p>

<p>If you could please also take a look through the style guidelines (which are mostly complete), and make those changes, it would be helpful. :)  <a href="https://github.com/mlpack/mlpack/wiki/StyleGuidelines">https://github.com/mlpack/mlpack/wiki/StyleGuidelines</a></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/430#issuecomment-99961850">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFBSX6LJZ0lkEYdWhNNxxQyVlDdsYks5oG5_JgaJpZM4D7GAx.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/430#issuecomment-99961850"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>