Oh, I have found another major problem with the script. The argparse Python module is only available for Python version 3.2 or higher. Therefore for most people this script will not be usable. For compatibility, I try to make sure that relax will run on Python versions 2.3 all the way to the currently highest version of 3.3 (http://www.nmr-relax.com/download.html). I have all version of Python installed on my Linux system for this testing (by all, I mean all including Python 0.1, 0.2, etc). You can see this in the devel_scripts/python_multiversion_test_suite.py script which comes with relax. Some people are forced to use old RHEL Linux distributions in which they are stuck with Python 2.5. So it would be useful to have the script functional for them too. I would suggest reverting to the older optparse module which is available on all systems. From the way you use the argparse module, it should be extremely easy to convert to optparse. The usage in the script looks almost identical to relax's use of the optparse.OptionParser() object in the arguments() method of the relax.py base module. There doesn't appear to be anything in the grace2images.py script which requires the newer module. Cheers, Edward On 19 June 2013 10:43, Edward d'Auvergne <edward@xxxxxxxxxxxxx> wrote:
Hi Troels, I would suggest a simple new patch to make your script executable. This only requires two lines (one import and one for changing the file mode). You can see how it is done by opening the pipe_control/palmer.py file and looking at the very last line of the create() function. Cheers, Edward On 18 June 2013 17:31, Edward d Auvergne <NO-REPLY.INVALID-ADDRESS@xxxxxxx> wrote:Update of bug #20916 (project relax): Status: None => Fixed Open/Closed: Open => Closed Release: Repository: trunk => Branches Operating System: Microsoft Windows => All systems _______________________________________________________ Follow-up Comment #6: This bug should have been a support request. The feature has been implemented by Troels and has now been committed. It has been applied to the relax_disp branch as it is specific to that branch and cannot be applied to the trunk. This was heavily discussed in the messages of the thread: http://thread.gmane.org/gmane.science.nmr.relax.devel/3953 _______________________________________________________ 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 relax-devel@xxxxxxx 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