[prev in list] [next in list] [prev in thread] [next in thread] 

List:       relax-devel
Subject:    Re: [task #6847] The Bieri graphical user interface.
From:       "Edward d'Auvergne" <edward () domain ! hid>
Date:       2010-05-31 11:54:56
Message-ID: AANLkTil3lAIZgUv-sgsJn8AYpuGpyvAz62PNkE93yRd3 () domain ! hid
[Download RAW message or body]

Hi,

Ok, I'll go through this patch bit by bit.  I can apply one patch with
all changes but, if it introduces bug, then the whole lot must go!
You will then have to salvage the patch somehow.  This might be more
of an issue for the dauvergne_protocol code than for the NOE analysis.
 I will also need a commit message from you detailing all these
unrelated changes.  Firstly, sorry, I didn't notice that the mc_num
arg is not actually used - that deletion is fine.  Only 2 things
remain:

1)  The "@type seq_args" epydoc documentation is missing and will
break the API documentation system.

2)  See my previous email about the heteronuc_pdb arg.

The rest is perfectly fine!  The patch applies cleanly.  So there is
not much to change here.  Sorry again for all the problems this is
causing, but we just need to be careful not to introduce bugs or
restrictions for relax users.

Cheers,

Edward


On 31 May 2010 03:18, Michael Bieri <NO-REPLY.INVALID-ADDRESS@gna.org> wrot=
e:
>
> Follow-up Comment #53, task #6847 (project relax):
>
> Hi Edward
>
> I applied your suggestions for the noe.py script. But I will send them as=
 one
> patch. I realy would like to pursuit as I will be in holuday the next 2
> weeks.
>
> Short comments to your comments on the noe.py script
> (https://mail.gna.org/public/relax-devel/2010-05/msg00006.html):
>
> I renamed filename to output_file.
>
> The argument mc_num is deleted, as it is not used in this script.
>
> The '@keyword heteronuc:' epydoc is added.
>
> I added another argument: neteronuc_pdb. This is to define the name of th=
e
> heteronucleus in the pbd file. This is also used in reading the sequence =
from
> the pdb file.
>
> I hope that's about it.
>
> If you are fine with these changes, I will modify them in the other scrip=
ts
> as well.
>
> Cheers
> Michael
>
>
>
>
> (file #9222)
> =A0 =A0_______________________________________________________
>
> Additional Item Attachment:
>
> File name: patch =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0Size:=
7 KB
>
>
> =A0 =A0_______________________________________________________
>
> Reply to this item at:
>
> =A0<http://gna.org/task/?6847>
>
> _______________________________________________
> =A0Message sent via/by Gna!
> =A0http://gna.org/
>
>
> _______________________________________________
> relax (http://nmr-relax.com)
>
> This is the relax-devel mailing list
> relax-devel@domain.hid
>
> 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
>


[prev in list] [next in list] [prev in thread] [next in thread]