failed unittest in py-kdtree_test.py

Martin SCHREIBER schreiberx at googlemail.com
Sun Dec 6 12:20:26 UTC 2009


hi!

On Sun, 2009-12-06 at 12:43 +0800, Paul Harris wrote:
> ...

>         Just leave find_within_range in place, and have it be a simple
>         one
>         line function calling one of the new functions. It is pretty
>         much
>         impossible to use the class without looking in the header
>         file, so it
>         will be crystal clear to everybody what is going on when using
>         the new
>         release. Now if the plan is to not support abs distance, then
>         you
>         should kill the function rather than changing the systematics.

i like to suggest that a #warning preprocessor message is generated when
the function find_within_range is called. the message should be
something like "this function is marked as deprecated and removed in the
next release. find_within_range_manhattan is a replacement for this
function".

ciao,
  martin


ps: i also didn't know, that this function isn't using the euclidian
distance for selection as i used libkdtree :-(

> 
> thanks for your comments :)
> Paul
> 
> 
> _______________________________________________
> libkdtree-devel mailing list
> libkdtree-devel at lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/libkdtree-devel


-- 

http://www.martin-schreiber.info

| Martin Schreiber
| Enzianstrasse 7, S 104
| 85748 Garching bei Muenchen
| GERMANY




More information about the libkdtree-devel mailing list