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