mailRe: [sr #3071] Implementation of Tollinger/Kay dispersion model (2001)


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

Header


Content

Posted by Troels Emtekær Linnet on September 02, 2013 - 16:13:
So I change ka to kA in specific_analysis\relax_disp\api.py

Do you know if this will make conflicts with other system tests?

Best


Troels Emtekær Linnet


2013/9/2 Edward d'Auvergne <edward@xxxxxxxxxxxxx>
Hi,

Maybe it would be better to change everything to kA?  In the
lib.dispersion code, I even call this parameter k_AB.  In the manual,
I have used kAB, where AB are subscripted.  So for consistency, I
think we should change everything everywhere!  Maybe the best would be
the k_AB parameter in the code?  The reverse rate would then be called
k_BA.  This would be confused with the kex,B = k_BA + k_AB rate, or kB
~= kex,B, but if kB was used like kA is, then this would be a mess.
What do you think?

Regards,

Edward



On 2 September 2013 15:16, Troels E. Linnet
<NO-REPLY.INVALID-ADDRESS@xxxxxxx> wrote:
> Follow-up Comment #15, sr #3071 (project relax):
>
> Renamed kA parameter to ka, to keep to already established parameter naming in
> relax.
>
> Patch applied.
>
> (file #18829)
>     _______________________________________________________
>
> Additional Item Attachment:
>
> File name: 0001-Renamed-kA-parameter-to-ka-to-use-the-same-parameter.patch
> Size:9 KB
>
>
>     _______________________________________________________
>
> Reply to this item at:
>
>   <http://gna.org/support/?3071>
>
> _______________________________________________
>   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


Related Messages


Powered by MHonArc, Updated Mon Sep 02 17:00:07 2013