<br><br><div class="gmail_quote">On Fri, May 6, 2011 at 12:12 AM, Derek Homeier <span dir="ltr"><<a href="mailto:derek@astro.physik.uni-goettingen.de">derek@astro.physik.uni-goettingen.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
On 5 May 2011, at 22:53, Derek Homeier wrote:<br>
<br>
>><br>
>> However, the problem that ndmin is supposed to address is not fixed<br>
>> by the current implementation for the rc.  Essentially, a single-<br>
>> row, multi-column file with ndmin=2 comes out as a Nx1 array which<br>
>> is the same result for a multi-row, single-column file.  My feeling<br>
>> is that if we let the current implementation stand as is, and<br>
>> developers use it in their code, then fixing it in a later release<br>
>> would introduce more problems (maybe the devels would transpose the<br>
>> result themselves or something).  Better to fix it now in rc with<br>
>> the two lines of code (and the correction to the tests), then to<br>
>> introduce a buggy feature that will be hard to fix in future<br>
>> releases, IMHO.<br>
>><br>
>> Looks okay, and I agree that it's better to fix it now. The timing<br>
>> is a bit unfortunate though, just after RC2. I'll have closer look<br>
>> tomorrow and if it can go in, probably tag RC3.<br>
>><br>
>> If in the meantime a few more people could test this, that would be<br>
>> helpful.<br>
>><br>
>> Ralf<br>
><br>
> I agree, wish I had time to push this before rc2. I could add the<br>
> explanatory comments<br>
> mentioned above and switch to use the atleast_[12]d() solution, test<br>
> that and push it<br>
> in a couple of minutes, or should I better leave it as is now for<br>
> testing?<br>
<br>
</div>Quick follow-up: I just applied the above changes, added some tests to<br>
cover Ben's test cases and tested this with 1.6.0rc2 on OS X 10.5<br>
i386+ppc<br>
+ 10.6 x86_64 (Python2.7+3.2). So I'd be ready to push it to my repo<br>
and do<br>
my (first) pull request...<br>
<div><div></div><div class="h5"></div></div></blockquote><div class="h5"> <br>Go ahead, I'll have a look at it tonight. Thanks for testing on several Pythons, that definitely helps.<br><br>Ralf <br></div></div><br>