search

thesamet / kdtree-scala

KDTree and KDTreeMap implementation in Scala
Apache License 2.0
33 stars 13 forks source link

Update KDTreeTest.scala #7

Closed 123avi closed 9 years ago

123avi commented 9 years ago

Add some tests for a point with sequence of features (need to add some more )

thesamet commented 9 years ago

Thanks for sending these commits. It looks great. I added a few requests for style changes to be consistent with the rest of this library.

123avi commented 9 years ago

Ok , I made the changes for all your comments as requested. if you want you can also reach me at 123avi@gmail.com . BTW I really like your work with the type system . Cheers Avi

2015-09-05 18:10 GMT+03:00 Nadav Samet notifications@github.com:

Thanks for sending these commits. It looks great. I added a few requests for style changes to be consistent with the rest of this library.

— Reply to this email directly or view it on GitHub https://github.com/thesamet/kdtree-scala/pull/7#issuecomment-137963684.