<blockquote>
<p>It's only needed in the constructor</p>
</blockquote>

<p>epsilon is my mistake, I did not check the codes carefully.The original codes try to initialize "non-existent epsilon", so the compilation can not pass</p>

<blockquote>
<p>I avoided the initialization list in favor of readability</p>
</blockquote>

<p>In this case, you have to declare b as non-const, because const data member must be initialize through the initialization list, else the compilation will fail.I am favor readability too:), maintainability <br>
is very important for long term development, In my humble opinion, codes quality of mlpack are quite well. </p>

<blockquote>
<p>but maybe it doesn't make a huge difference</p>
</blockquote>

<p>I think this is based on the taste of the programmers, if the initialization process is too complex, we could hide it in data member.</p>

<p>By the way, is it ok to change the parameters of Train to const(as the channel logs of 18/20/2015 suggest)?Thanks</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/462#issuecomment-149040186">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFH-up-c80Oi2hP-gPSPBr8BYnfftks5o8-hOgaJpZM4GQ1U6.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/462#issuecomment-149040186"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>