Hi, This looks good, though I would suggest adding many comments to explain what is happening. I can clearly see what is happening, but most users would require very basic English descriptions to understand. I would suggest adding to the top:
#! /usr/bin/env python Then add a docstring for the file explaining what it is. I would also suggest to make it executable. This will work for Linux and Mac people, but for Windows users there should be in the docstring for the script some text saying that you run the file by typing 'python this_script.py' or any other way to run it. One last thing, this report would be much better as a support request or created as a task. Cheers, Edward On 17 June 2013 16:53, Troels E. Linnet <[email protected]> wrote: > URL: > <http://gna.org/bugs/?20916> > > Summary: Suggestion for python script for PNG/EPS/SVG > conversion of grace files > Project: relax > Submitted by: tlinnet > Submitted on: Mon 17 Jun 2013 02:53:02 PM GMT > Category: relax's source code > Severity: 1 - Wish > Priority: 5 - Normal > Status: None > Privacy: Public > Assigned to: None > Originator Name: > Originator Email: > Open/Closed: Open > Discussion Lock: Any > Release: Repository: trunk > Operating System: Microsoft Windows > > _______________________________________________________ > > Details: > > Suggestion for python script for the conversion of grace files .agr to > PNG/EPS/SVG. > > Could be written together with the grace files. > > > > > > _______________________________________________________ > > File Attachments: > > > ------------------------------------------------------- > Date: Mon 17 Jun 2013 02:53:02 PM GMT Name: convert.py Size: 2kB By: > tlinnet > > <http://gna.org/bugs/download.php?file_id=18102> > > _______________________________________________________ > > Reply to this item at: > > <http://gna.org/bugs/?20916> > > _______________________________________________ > Message sent via/by Gna! > http://gna.org/ > > > _______________________________________________ > relax (http://www.nmr-relax.com) > > This is the relax-devel mailing list > [email protected] > > To unsubscribe from this list, get a password > reminder, or change your subscription options, > visit the list information page at > https://mail.gna.org/listinfo/relax-devel _______________________________________________ relax (http://www.nmr-relax.com) This is the relax-devel mailing list [email protected] To unsubscribe from this list, get a password reminder, or change your subscription options, visit the list information page at https://mail.gna.org/listinfo/relax-devel
