<p>In <a href="https://github.com/mlpack/mlpack/pull/694#discussion_r70161999">src/mlpack/core/data/map_policies/missing_policy.hpp</a>:</p>
<pre style='color:#555'>> + {
> + // Nothing to initialize here.
> + }
> +
> +
> + template <typename MapType>
> + mapped_type MapString(const std::string& string,
> + const size_t dimension,
> + MapType maps,
> + std::vector<Datatype>& types)
> + {
> + // If this condition is true, either we have no mapping for the given string
> + // or we have no mappings for the given dimension at all. In either case,
> + // we create a mapping.
> + const double NaN = std::numeric_limits<double>::quiet_NaN();
> + if (missingSet.count(string) != 0 &&
</pre>
<p>Ah, I didn't think of that. I agree now, putting everything from <code>missingSet</code> into <code>maps</code> is not the right thing.</p>
<p>I agree, too, Boost spirit seems really cool, I need to play with the code in your PR to see if we can preserve decent compile times.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<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/files/a8818316a04506530e2269a2e0a32ba2f6a1c83b#r70161999">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFEM9XEPV1KCrkdBlUDJSWxySXcfvks5qTxuNgaJpZM4I07W-">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFKZoK4_X10iLoab9PeulFF_xKsNJks5qTxuNgaJpZM4I07W-.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/files/a8818316a04506530e2269a2e0a32ba2f6a1c83b#r70161999"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>