[SciPy-Dev] np.savetxt: apply patch in enhancement ticket 1079 to add headers?
Fri Jun 4 13:08:15 CDT 2010
On 06/04/2010 11:03 AM, Skipper Seabold wrote:
> On Fri, Jun 4, 2010 at 11:49 AM, Stefan<firstname.lastname@example.org> wrote:
>> Dear all,
>> as a consequence of our discussion, we developed a patch (attached to
>> ticket 1079), which implements some of the features discussed here.
>> We concentrated on comments and the header. Please have a look at the
>> patch. We are looking forward to hearing your opinion and suggestions,
>> and whether you see any problems, which could prevent it from entering the
>> official release.
> Link: http://projects.scipy.org/numpy/ticket/1079
> One comment. Maybe you can add in the notes that the comment keyword
> can be used to write a header and still preserve compatibility with
> loadtxt. This wasn't obvious to me at first, though maybe that's just
> Other than that I think it looks like a good first effort towards
> making this a better function and I appreciate the attention here.
>> We agree with Bruce that the format string should be inferred from the
>> data type of the array. Yet, we believe that this point should be
>> addressed in a different patch focussing on that topic.
>> Also we noted that there is no error checking, when an array of dimension
>> larger 2 is handed to np.savetxt, which may be implemented easily.
>> Stefan& Christian
>> SciPy-Dev mailing list
> SciPy-Dev mailing list
For the sake of similarity to loadtxt keywords (because loadtxt has them
and changing those is harder than adding new ones to savetxt):
1) 'comment_character' should be 'comments'
2) instead of 'comment' perhaps use 'preamble'
Thanks for doing the patch so quickly!
More information about the SciPy-Dev