-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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.
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
User rvalue references to set a given reference tree in RASearch, RangeSearch, FastMKS, and DualTreeBoruvka #773
Comments
Hi Marcos I would like to take up this issue. I am reading #765 to understand the issue. I am completely new to mlpack and machine learning. I have gone through the getting involved page. |
@itabhiyanta I already have a PR (#799) in motion for the #765 is a good place to start. You might want to take a look at all the discussion there. They are helpful in understanding what is being implemented. All the best :) |
@akanuraj200 I would like to try RASearch and RangeSearch as I am new here, and would like to start with something labeled easy. Are you working on it? Or can I? |
Hi
I haven't started so you may pick it up.
Rohit
…On Fri, Dec 16, 2016 at 7:37 AM, aneesh2312 ***@***.***> wrote:
@akanuraj200 <https://github.com/akanuraj200> I would like to try
RASearch and RangeSearch as I am new here, and would like to start with
something labeled easy. Are you working on it? Or can I?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#773 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AHrFxtFlSYBm_ujbnoY26Hs3lWtQn6a9ks5rIjG4gaJpZM4Jv8gj>
.
|
Hello @aneesh2312 |
Hi, |
@abhinavrai I am working on it.. |
@aneesh2312 are you still working on it ?If no i would like to work on this issue .Thank You!! |
@Lakshya1605 I have done RAsearch, but I never got time to finish RangeSearch.. You can take RangeSearch... |
@aneesh2312 Could you please give an overview of what is expected to be done . |
Update the code to include changes similar to the ones included in the PR: #765
The text was updated successfully, but these errors were encountered: