mailRe: structure.vector()


Others Months | Index by Date | Thread Index
>>   [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Header


Content

Posted by Edward d'Auvergne on February 17, 2009 - 22:36:
On Fri, Feb 6, 2009 at 6:38 PM, Sébastien Morin
<sebastien.morin.1@xxxxxxxxx> wrote:
Hi,

I was looking at the code for function 'vectors()' in both
'prompt/structure.py' and 'generic_fns/structure/main.py' and saw some
possible problems...

============
1.
============
The default arguments for 'prompt/structure.vectors()' are:

       attached='H'
       spin_id=None
       model=None
       verbosity=1
       ave=True
       unit=True

while, for 'generic_fns/structure/main.vectors()', they are:

       attached=None
       spin_id=None
       model=None
       verbosity=1
       ave=True
       unit=True

The attached argument has two different default values ? Is that normal ?

This is perfectly normal.  The vectors() function in
generic_fns.structure.main is supposed to be generic for calculating
the vector between 2 atoms.  The structure.vectors() user function is
used more often in relax scripts for getting the XH bond vector where
X is represented by the spin container and H is the attached atom.
'H' probably the most generic name for the attached proton, and this
works nicely for protein NH work.  Note this default value is used
simply as a suggestion and if a user forgets to set this correctly,
they will quickly find out with all the warning messages.


============
2.
============
There seems to be an error in the docstring example for
'prompt/structure.vectors()' where the argument 'attached' is replaced
by 'proton'...

      relax> structure.vectors(proton='H', spin_id='@N')
      ...
      relax> structure.vectors(proton='HN', spin_id='@N')
      ...
      relax> structure.vectors(proton='HA', spin_id='@CA')


Should this be corrected ?

Yes!  The 'attached_atom' argument used to be called 'proton' until I
generalised the user function.  So this is just a out of date
docstring.

Cheers,

Edward



Related Messages


Powered by MHonArc, Updated Tue Feb 17 23:00:25 2009