Could you create a bug report for the missing relax_disp.relax_time menu entry anyway? We can then tackle such problems later.
Regards, Edward On 28 August 2013 11:52, Troels Emtekær Linnet <[email protected]> wrote: > 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
