[mlpack-git] [mlpack/mlpack] Vantage point tree and HRectBound improvements (#760)

[Ryan Curtin]

> @@ -35,6 +36,8 @@ class HollowBallBound
>    math::RangeType<ElemType> radii;
>    //! The center of the ball bound.
>    arma::Col<ElemType> center;
> +  //! The center of the hollow.
> +  arma::Col<ElemType> hollowCenter;

I thought to myself, 'surely, holding two centers, which can double the number of distance computations or more for a single score, can't be as fast as only holding one, even if the bounds end up being a little looser'.  But after I tried implementing it, and removing the `hollowCenter` member, the runtimes were always about 10% slower.  So it seems that `hollowCenter` is the right way to go here.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/760/files/55e15792e4dc2135c6e3421c5dae1322d07b0dcc#r74709867
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160814/7f352731/attachment.html>