[mlpack-git] [mlpack/mlpack] R+ and R++ trees implementation (#699)

[lozhnikov]

> +      if (node->Dataset().col(node->Point(sorted[i].n))[k] > highBound2[k])
> +        highBound2[k] = node->Dataset().col(node->Point(sorted[i].n))[k];
> +    }
> +  }
> +
> +  // Evaluate the cost of the split i.e. calculate the total coverage
> +  // of two resulting nodes.
> +
> +  ElemType area1 = 1.0, area2 = 1.0;
> +  ElemType overlappedArea = 1.0;
> +
> +  for (size_t k = 0; k < node->Bound().Dim(); k++)
> +  {
> +    area1 *= highBound1[k] - lowerBound1[k];
> +    area2 *= highBound2[k] - lowerBound2[k];
> +  }

Ah, I forgot about `operator|=()`. 
`overlappedArea` is equal to 0. This variable is not needed 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/699/files/e165d759f9ae612b9965f70fbbf8abdb19dc8d07#r69029630
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160629/0012ffff/attachment.html>