-
Notifications
You must be signed in to change notification settings - Fork 84
Use k-d tree to speedup nearest-neighbor lookup #7
Comments
@joshhting how's this coming along? |
Slowly and steadily, it's mostly me being confused by how C++ works. I'm trying to make the project with flann, the k-d tree submodule that I'm using, and I'm currently sorting through errors in the linking stage because apparently some of the files in the flann submodule are C++ headers with C implementation files. |
Ok, well let me know if you want help with anything. If you want to push your code to GitHub, I can take a look at the linker errors. |
Ok I pushed some code to the flann branch on my fork. When I run make, I get an undefined reference error which I'm not sure how to solve. |
Ok, I think I've got this working. See commit 5e53662 on the justin/flann-compile-fix branch. The main issue was that the flann library wasn't getting compiled by CMake. CMake only builds the c++ files that you tell it to in the CMakeLists.txt file. You had an |
I'm still having trouble compiling it, but now it's because it gets stuck at 14% trying to compile flann.cpp.o. Do you know what could cause make to freeze up here? |
Do you get a compile error or does it just hang forever? |
I just tried compiling again on my computer and it worked, although it took forever for it to compile flann.cpp. |
Hmm, it just hangs forever for me when it's compiling flann.cpp. I guess I'll just leave it on for longer for now. |
Just a heads up, theres something off about the flann submodule, It dosent seem to be in the history of the submodule on github. I think its a dangling commit?
|
Yeah I noticed that too, not sure what happened there... I just |
I just pushed an updated version of that submodule checkout to my branch. If you pull from |
Now I get the following error when I run make (I pulled from
|
http://en.wikipedia.org/wiki/K-d_tree
The text was updated successfully, but these errors were encountered: