[mlpack-git] [mlpack/mlpack] Random projection trees (#726)
notifications at github.com
Sun Aug 7 23:29:20 EDT 2016
@lozhnikov yes! I agree that there is no error. I meant to say that it was a bit confusing not using a hyperplane there.... because Random Projection Trees seems to be based on splitting hyperplanes. But, I am sure they have their reasons!
It becomes a bit difficult to generalize some concepts when we implement many differents flavours! :)
So, I think we can consider both PR separately, and once they are merged with the master branch we can see if it is possible to unify both implementations of the splitting algorithms, to avoid duplicated code. Would you agree?
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the mlpack-git