Add inrange2() to find points between two radiuses#52
Add inrange2() to find points between two radiuses#52chuong wants to merge 4 commits intoKristofferC:masterfrom
Conversation
|
Some tests in test_monkey.jl failed, but none of these relate to inrange2() implementation. Somehow I don't have these fail tests on my computer. |
|
Thanks for the PR! Do you think perhaps Also, would be nice with a few more tests on a larger set of points where you compare that you get the same result for all the different type of trees. |
|
Thanks. Your suggestions are all possible. I am struggling with naming the new function. On Wikipedia, new function is similar to |
This is useful when searching for points at a radius from fixed position with a tolerance. This is equivalent to, but more efficient than, using two inrange() operations for two different radiuses and then finding point that belong to only the larger radius.