<br><br><div class="gmail_quote">On Sat, Jun 13, 2009 at 12:29 PM, David Cournapeau <span dir="ltr"><<a href="mailto:cournape@gmail.com">cournape@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Sun, Jun 14, 2009 at 3:00 AM, Charles R<br>
<div class="im">Harris<<a href="mailto:charlesr.harris@gmail.com">charlesr.harris@gmail.com</a>> wrote:<br>
><br>
><br>
</div><div class="im">> On Sat, Jun 13, 2009 at 7:46 AM, David Cournapeau<br>
> <<a href="mailto:david@ar.media.kyoto-u.ac.jp">david@ar.media.kyoto-u.ac.jp</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>>    I have cleaned up a bit the code, and would like to suggest the<br>
>> inclusion of a neighborhood iterator for numpy. Stéfan took a look at it<br>
>> already, but it needs more eyeballs. It is a "subclass" of<br>
>> PyArrayIterObject, and can be used to iterate over a neighborhood of a<br>
>> point (handling boundaries with 0 padding for the time being).<br>
>><br>
>> <a href="http://codereview.appspot.com/75055/show" target="_blank">http://codereview.appspot.com/75055/show</a><br>
>><br>
>> I have used it to replace the current for code correlateND in<br>
>> scipy.signal, where it works quite well (I think it makes the code more<br>
>> readable in that case).<br>
><br>
> Some nitpicks:<br>
><br>
> 1) The name neigh sounds like a horse. Maybe region,  neighborhood, or<br>
> something similar would be better.<br>
<br>
</div>Neighborhood makes the name quite long - maybe region would be better,<br>
although region does not imply the notion of "contiguity" ?<br>
<div class="im"></div></blockquote><div><br>It does make it long, which is an inconvenience to the coder. But I think it is easier on the reader. Hmm... a thesaurus also suggests zone, area, and vicinity.<br><br>Chuck<br>
</div><br></div><br>