<p>In <a href="https://github.com/mlpack/mlpack/pull/694#discussion_r69909687">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>We could avoid this check if you added everything from <code>missingSet</code> to <code>maps[dimension]</code> in the constructor. That might give some noticeable speedup (I am not sure about that).</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#r69909687">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFPjiS1xW8g_InjKSt9TR4TolCa4dks5qTQSvgaJpZM4I07W-">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFNnxY1WJVd-TQrgOrwf5gjsv2qN2ks5qTQSvgaJpZM4I07W-.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#r69909687"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>