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
