failed unittest in py-kdtree_test.py

[Jason Remillard]

Hi,

I am one of the users that has been caught by the find_within_range
functions :-)

Breaking peoples code is kind of drastic. A simple comment above each
find functions would have prevented any confusion on my part.

Just add in the new functions (I assume they look something like this)

find_within_range_abs
find_within_range_manhattan
find_within_range_euclidean

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.

Thanks
Jason.

On Fri, Dec 4, 2009 at 11:28 PM, Yaroslav Halchenko
<debian at onerussian.com> wrote:
>
> On Sat, 05 Dec 2009, Paul Harris wrote:
>>      grep-dctrl -sPackage -F Build-Depends libkdtree++-dev
>>      /var/lib/apt/lists/*_debian_dists_sid_main_source_Sources
>>      on a Debian system (package dctrl-tools)
>>    I didn't find anything on my system, so thats good I think.
> yeap -- for Debian -- break of API would not have major issues (that is
> what I implied before) since none package build-depends on it
>
>>    And if we can make the software break at the compile stage (ie before
>>    packaging and shipping), then the end-user need never see an error
>>    message or deprecated warning.
> Agree 100% -- lets make grandma safe! ;-)
>
> --
>                                  .-.
> =------------------------------   /v\  ----------------------------=
> Keep in touch                    // \\     (yoh@|www.)onerussian.com
> Yaroslav Halchenko              /(   )\               ICQ#: 60653192
>                   Linux User    ^^-^^    [175555]
>
>
>
> _______________________________________________
> libkdtree-devel mailing list
> libkdtree-devel at lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/libkdtree-devel
>