Hi Edward. Well, I would be interested in looking into this.
But at the moment, I am trying to finish the Tollinger/Kay model and implement the test-data set. And then I would have to mode forward to push some data through relax. So, even though I could learn something from making the corrections, I will have to prioritize my time at the moment. I will though fix the NMRPipe bug. :-) Best Troels Troels Emtekær Linnet 2013/8/28 Edward d'Auvergne <[email protected]>: > It should be possible to do this, but note that the 'time' name error > has absolutely nothing to do with this. There are three separate bugs > here: > > - RelaxError: The Value element is read only, cannot set the value 'auto'. > - NameError: global name 'time' is not defined. > - Setting the relaxation time period for an experiment if the spectra > loading wizard is cancelled. All the user functions of the wizard > should be presented. > > The first two can be part of the current bug #21076 > (http://gna.org/bugs/?21076). The last should be its own bug report. > For testing this, the > test_suite/shared_data/dispersion/Hansen/relax_results/final_state.bz2 > file can be opened in the GUI, the pipe editor window opened (via the > View menu for example), the base pipe right clicked on, and then the > 'Associate with a new auto-analysis' entry clicked on. Right clicking > on any of the spectra list entries shows that the > relax_disp.relax_time user function is not present in the pop-up menu. > When creating a new report, please add a link to this message in the > Gmane thread (go to > http://thread.gmane.org/gmane.science.nmr.relax.devel/4417, click on > the correct message, then click on <<< [thread] >>> down the bottom to > go to the correct link). > > For me, the fixes are trivial. This is because I know the code in > question very well. But I would like to give you the opportunity to > solve these problems as it will be a great teaching experience. You > will become much more familiar with the code layout of relax and > afterwards you will be in a much better position for being a relax > developer, as you will be better able to solve problems in your own > code. You will also be in a better position to solve problems in the > GUI or other UIs as your students encounter them - as students always > do things in ways you do not expect and find things you would never > encounter yourself - and then you could make fixes on the spot for > them and then push the fixes upstream. > > Cheers, > > Edward > > > On 28 August 2013 09:49, Troels Emtekær Linnet <[email protected]> wrote: >> How would you add to the "right-click" menu, the possibility to add >> Relaxation dispersion CPMG constant time delay T (in s). >> to the experiment? >> >> >> >> Troels Emtekær Linnet >> >> >> 2013/8/28 Edward d'Auvergne <[email protected]>: >>> Hi, >>> >>> The NameError with 'time' is a clear bug. It is only in the >>> relax_disp branch. I can again let you try to find the solution, if >>> you like. And again, it is a one line fix - I know the solution >>> already. This is not an issue of a CPMG constant delay time, but just >>> a simple typo. >>> >>> Regards, >>> >>> Edward >>> >>> >>> On 27 August 2013 21:03, Troels Emtekær Linnet <[email protected]> wrote: >>>> How about, the following error message, >>>> when trying to define the CPMG frequence. >>>> >>>> NameError: global name 'time' is not defined >>>> >>>> I guess there needs to be added the CPMG constant delay time for the >>>> spectra. >>>> >>>> But, when you right click, there is no shortcut to add the >>>> # Relaxation dispersion CPMG constant time delay T (in s). >>>> relax_disp.relax_time(spectrum_id=current_id, time=time_T2) >>>> >>>> And should time_T2 not appear from the list in the GUI? >>>> >>>> >>>> >>>> >>>> Troels Emtekær Linnet >>>> >>>> >>>> 2013/8/27 Edward d'Auvergne <[email protected]>: >>>>> Hi Troels, >>>>> >>>>> If you like, I'll first let you try to find the source of the problem. >>>>> Note it is in the relax trunk. The key part is in the error message, >>>>> specifically the 'read only' part. If you would like hints, just ask. >>>>> >>>>> Regards, >>>>> >>>>> Edward >>>>> >>>>> >>>>> >>>>> >>>>> On 27 August 2013 17:10, Edward d Auvergne >>>>> <[email protected]> wrote: >>>>>> Update of bug #21076 (project relax): >>>>>> >>>>>> Category: relax's source code => GUI >>>>>> Status: None => Confirmed >>>>>> Release: Branches => Repository: trunk >>>>>> Operating System: GNU/Linux => All systems >>>>>> >>>>>> >>>>>> _______________________________________________________ >>>>>> >>>>>> Reply to this item at: >>>>>> >>>>>> <http://gna.org/bugs/?21076> >>>>>> >>>>>> _______________________________________________ >>>>>> Message sent via/by Gna! >>>>>> http://gna.org/ >>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> relax (http://www.nmr-relax.com) >>>>>> >>>>>> This is the relax-devel mailing list >>>>>> [email protected] >>>>>> >>>>>> 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 _______________________________________________ relax (http://www.nmr-relax.com) This is the relax-devel mailing list [email protected] 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
