[Numpy-discussion] Code Freeze for NumPy 1.7
Travis Oliphant
travis at continuum.io
Sun Jul 15 23:12:05 EDT 2012
This looked like an interesting one for sure. I can't look at the PR right now for some reason (Github gave me a 500 error). I know there were some comments, though.
-Travis
On Jul 15, 2012, at 9:52 PM, Frédéric Bastien wrote:
> Hi,
>
> there is a PR that I think could be merged before the relase:
>
> https://github.com/numpy/numpy/pull/326
>
> It is the addition of the inplace_increment function. It seam good,
> but I can't review it enough as it use many numpy internal that I
> never used or looked at. But the tests seam to cover all cases and it
> don't change current functions. So it should not have any side effect
> problems.
>
> This was a feature frequently requested.
>
> Fred
>
> On Sun, Jul 15, 2012 at 10:40 PM, Travis Oliphant <travis at continuum.io> wrote:
>>>
>>> Bump for: https://github.com/numpy/numpy/pull/351
>>>
>>>
>>> As requested by njsmith, I gave a more detailed explanation and asked
>>> the list for input at:
>>> http://www.mail-archive.com/numpy-discussion@scipy.org/msg38306.html
>>>
>>> There was one qualified negative reply and nothing (yet) further. I'd
>>> appreciate if some other devs could weigh in.
>>
>> I can see the point of your proposal, but I don't think we can just change the behavior of genfromtxt quite like this. I'm not an expert on the genfromtxt code, so I don't know if I exactly understand what is changing and what is staying the same.
>>
>>> From the look of the patch, it looks like you are changing the interpretation so that whereas headers used to be allowed in the "first-line" of the comment, they would no longer be allowed like that. I don't think we can do that, because it breaks code for someone without a path for change.
>>
>> Now, we could add another keyword (headers=True, for example), that interpreted the first non-commented line as the header line. Something like that has a much higher chance of getting accepted from my perspective.
>>
>> Best,
>>
>> -Travis
>>
>>
>>
>>
>>>
>>>
>>> Thanks,
>>> --
>>> Paul Natsuo Kishimoto
>>>
>>> SM candidate, Technology & Policy Program (2012)
>>> Research assistant, http://globalchange.mit.edu
>>> https://paul.kishimoto.name +1 617 302 6105
>>> _______________________________________________
>>> NumPy-Discussion mailing list
>>> NumPy-Discussion at scipy.org
>>> http://mail.scipy.org/mailman/listinfo/numpy-discussion
>>
>> _______________________________________________
>> NumPy-Discussion mailing list
>> NumPy-Discussion at scipy.org
>> http://mail.scipy.org/mailman/listinfo/numpy-discussion
> _______________________________________________
> NumPy-Discussion mailing list
> NumPy-Discussion at scipy.org
> http://mail.scipy.org/mailman/listinfo/numpy-discussion
More information about the NumPy-Discussion
mailing list