vioshyvo / mrpt

Fast and lightweight header-only C++ library (with Python bindings) for approximate nearest neighbor search
MIT License
257 stars 47 forks source link

float64 queries should throw an error #10

Closed kylemcdonald closed 6 years ago

kylemcdonald commented 6 years ago

It looks like float64 queries are not supported. But the failure is silent, and mrpt still returns results. This created a really complicated debugging problem for me when comparing two datasets and i forgot that the datatype of one was float64. Here's an example of the issue:

https://gist.github.com/kylemcdonald/b9c6eb400ccf688457be6d57ade78c17

ejaasaari commented 6 years ago

Thanks!