Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Further generalization of trees to include ball trees #184

Closed
rcurtin opened this issue Dec 29, 2014 · 2 comments
Closed

Further generalization of trees to include ball trees #184

rcurtin opened this issue Dec 29, 2014 · 2 comments
Labels
s: fixed t: feature request
Milestone
mlpack 2.0.0

Comments

@rcurtin
Copy link
Member

rcurtin commented Dec 29, 2014

Reported by rcurtin on 3 Oct 42063795 20:34 UTC
There are several implementations of ball trees floating around in contrib/. We need to figure out how to make our own tree usage more generalized so that a user can either pass a BinarySpaceTree or a BallTree to a method.

Migrated-From: http://trac.research.cc.gatech.edu/fastlab/ticket/190
@rcurtin rcurtin self-assigned this Dec 29, 2014
@rcurtin rcurtin added this to the mlpack 1.1.0 milestone Dec 29, 2014
@rcurtin rcurtin removed their assignment Dec 30, 2014
@rcurtin rcurtin removed the C: mlpack label Jan 8, 2015
@rcurtin
Copy link
Member Author

rcurtin commented Jan 9, 2015

I think that this just needs testing. See also #349 .

@rcurtin
Copy link
Member Author

rcurtin commented Jul 15, 2015

Why is this still open? This was done a long time ago...

@rcurtin rcurtin closed this as completed Jul 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
s: fixed t: feature request
Projects
None yet
Milestone
mlpack 2.0.0
Development

No branches or pull requests
1 participant
@rcurtin