<p>Ok, it seems like everyone thinks using default DEBUG=OFF and PROFILE=OFF is a good idea, so I went ahead and merged commit <a href="https://github.com/mlpack/mlpack/commit/3fe0b72997697baf470ef25d959218cdb81d3c4a" class="commit-link"><tt>3fe0b72</tt></a> to merge this PR (I did it manually because there is weirdness in this PR and I did not want to try to merge it and break everything...).</p>

<p><a href="https://github.com/SterlingPeet" class="user-mention">@SterlingPeet</a>: thanks for the detailed response.  I enjoyed the article you linked to.  I think maybe at some point it may be worth changing to the model you suggested, but I am not sure that is a huge deal at the moment (maybe as the project grows that will be more necessary).  As it stands now, we do use feature branches, which only get merged when the code is tested and ready.  So I guess realistically master only contains code-that-will-be-released-but-maybe-isn't-quite-yet.  Anyway, I'll think about it, but it would be some amount of effort to change, so unless someone else wants to take the lead, I'm fine leaving it how it is now (until there is a later problem, perhaps).</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/662#issuecomment-230558362">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFMWIVzBNZdjw_jiQvVoMfZhRCQyhks5qSp87gaJpZM4Iq2lg">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFKtaOc6LTN-Y6f-lyx4JwZ06EENlks5qSp87gaJpZM4Iq2lg.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/662#issuecomment-230558362"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>