<p>The modified optimizer looks good, thanks for the effort. Once we merge the altered optimizer interface, we have to modify all layer that uses the optimizer interface. If you need some help with this step, don't hesitate to ask.</p>

<blockquote>
<p>mlpack 2.x still rely on libxml? Could we just use boost to replace it?</p>
</blockquote>

<p>You are right since mlpack 2.0.0 the libxml2 dependency was replaced with boost::serialization.</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/516#issuecomment-182643667">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFGe0l_cY7oMtrIhQSh-u5su86Ko9ks5pi8g3gaJpZM4HTzyk.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/516#issuecomment-182643667"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>